Select - Two additional classes

If possible, can You add two additional classes for the SELECT field?
Ex.
data-cls-selected for the item currently selected (now light Cyan)
and
data-cls-selection for the bar selection (now gray)
Ty!

Don't work?
https://codepen.io/TorakikiSan/pen/EeyWmp

selected item exists only for select with multiple, for the single select box - select-input

For example:

I hope I have explained better :-)

To change hover state for option:
<select data-role="select" data-cls-option="bg-red-hover fg-white-hover">
  <option value="1">Item 1</option>
  <option value="2">Item 2</option>
  <option value="3">Item 3</option>
</select>

I add attribute data-cls-option-active in 4.2.24, data-cls-selected I will not add


org settings: Move function out of setup function.

This just moves some functions out of _setup function because they are unnecessarily initialized at the setup time.  These functions were inside _setup function because in past we have property_types inside _setup which contains some translation string but those strings are removed so we don't have these function and JS objects inside _setup.
Also, there are some minor fixes for bugs in commits,
org settings: Make sync_realm_settings to be available at load time.
This moves sync_realm_settings function out of _setup so that
we can call settings_org.sync_realm_settings without opening
settings page atleast once.
This also fixes a minor bug in which if we have two tabs opened and
in one we haven't opened settings page at least once and in
another we change an org setting, then we get an exception in
the former tab because of the event as sync_realm_settings
isn't defined yet.
org settings: Fix real-time sync for realm_waiting_period_threshold.
This fixes a minor bug in which the value of input element of
realm_waiting_period_threshold don't get updated because in
set_create_stream_permission_dropdwon we don't change the
value of the input.
So, this minor refactor handles this more carefully.

Is there any commit needed to split/squashed further?

This is great, merged, thanks @shubhamdhama!  We might want to de-export some of the exported functions that aren't used elsewhere as a follow-up.


When plotting large dataSets other than timeseries data, zooming, panning and scrolling feature need to be added in a bar chart.

Is your feature request related to a problem? Please describe.
In a category x-axis chart, zooming/panning is not enabled if xaxis.categories is defined as String values.
Describe the solution you'd like
If in case the dataType is string and the entire data is huge than the interactive functionality should be enable.
Describe alternatives you've considered
Enable zooming/panle for function for each datatypes.
Additional context
Add any other context or screenshots about the feature request here.

When plotting large dataSets other than timeseries data, zooming, panning and scrolling feature need to be added mainly in bar chart.

Sorry, zooming/panning on category x-axis is not supported currently.
It might be added in the future release, no ETA.


Tab Protection

Description of the new feature/enhancement
Tab Protection to keep long running processes, development scripts from being accidently closed via close all button within the terminal.
Proposed technical implementation details (optional)
When you right click the tab, you have an option to "Protect" , "Pin", "Guard" or something like that to the specific tab you have open.
If you click the right most X, on the terminal app close all non "Protected" tabs
Pop up a toast or message asking if you would like to close the protected tab and or if multiple tabs . (I would have two buttons one for single tab, one for all)

Would something like read-only mode work for you/? That's a bit more heavy handed - that disables closing AND input.
If not, then this should probably be duped to #6549 (which is broadly tracking all the combinations of "prompt me before I close a {{thing}}")

The read only mode is interesting. I would say for now maybe #6549 makes some sense as its appear more like what I was shooting for.


How can i get prev/next month name

you give such a solution - but the script does not return these values.
// the month name- don't forget that you can do things like
// month.substring(0, 1) and month.toLowerCase() in your template
previousMonth: "April"
month: "May"
nextMonth: "June"
// the year that the calendar is currently focused on
year: "2013"

@krismo-pl can you please paste your code here? Any example would help us answer your question better.

I'm going to close this out, @krismo-pl you will have better luck on something like Stack Overflow for your question, as this issue list is only for monitoring bugs and browser problems.


Wheres the html code for opening it on website browser

If this ai town is legit did I make mistake in reading the basics? Like following it to them knowledge to actually have ais go-to the html code of ai town??

While running npm run dev you can visit http://localhost:5173/ai-town


Update create dataset from database modal layout

The layout has been updated for aesthetic reasons.
The functionality remains the same.
A small code bug has been fixed.
This PR addresses issue https://github.com/Kitware/minerva/issues/424
New layout:

Existing layout:

the new layout is much better thanks!

@Thank you for your suggestion in the first place.


Does docker swarm cluster support publisher / reader pattern

I am new to this events project, want to understand whether docker swarm cluster support publisher / reader pattern. I am looking for way working like this:

there is a docker cluster, take swarm cluster as example.
I can setup a client listener, which is connect to the cluster, my client can be notified automatically no matter what changes in the cluster, like new node joined, new images pulled, new container start/ stop, etc.
Does this event project support this?

@yangguoyk2015 This package is a generic event distribution package. So the answer is yes. Is there specific code to do exactly this within this package? No.
Please read the documentation to get a better idea of the use cases for this package.

@stevvooe , thanks for your quick response. Per my understanding, so far, in Docker projects, only the Docker registry has implemented the auto notification strategy, that's where this go-events project come from.
We have a desire that maybe we can build a small image, which can work together with Docker Engine or swarm, with this new image, it can collect all the changes (new nodes joined/ new containers, etc) in the docker cluster, and convert these changes into  a standard event, and broadcast these events to its subscribers, similar way as webhook, you can choose the interested events and set the target end-point URL. If we can make this, we can receive the latest notification and get the latest status in the cluster, instead we have to monitor the cluster status manually.
Let me know your comment. Thanks!

@yangguoyk2015 That sounds somewhat like the long term goal of this project. It would be great if you could give this package a try and let me know what you are missing. With the current docker engine, you would have to call the events API and then aggregate and distribute that. We may want to add to notifications to docker itself, but that is not planned. For your use case, I'd recommend running some sort of aggregation agent based on this package and work from there. If we find it may be better to implement that within docker, we can work from there.
Part of the next step here is porting over the plumbing for notifications.Endpoint. I'll probably try this week or next week, depending on my schedule. We also need the "inverse" of that, which accepts calls from remote HTTP service and dumps it into an event queue. We may also want to define sinks for commonly used queueing services, such as kafka, sqs and rabbitmq.
To provide you a little context, the https://github.com/docker/engine-api/issues/89#issuecomment-198570221 proposes some changes to the engineapi.Events call that I'd like to compatible with this dispatch package. We still need to define a Source that can be compatible with the Sink definition.

@stevvooe , thanks for your answer. I will try per your comment, will see how amazing Docker will be :).

@yangguoyk2015 I don't think your question is related to this repo. As far as I know, k8s is designed to be event driven, the only thing you need is watching etcd events for the change you are interested in (like container status changes etc).

Should this Issue be closed?


Error: sql/sqlclient: unknown driver "libsql+ws". See https://atlasgo.io/url

Hello I've been following the issue #1951, which was finally solved by  #2010 seemingly making local development with sqld possible. But I am still met with the Error: sql/sqlclient: unknown driver "libsql+ws". See https://atlasgo.io/url when I try to use atlas schema apply command against the local sqld instance. I am currently on v.0.15.0 of atlas.
I start my sqld process that listens on port 6969 with this:
  turso dev --db-file db-file --port 6969

My atlas hcl:
variable "destructive" {
  type    = bool
  default = false
}

variable "token" {
  type    = string
  default = getenv("TURSO_DB_AUTH_TOKEN")
}

env "dev" {
  url     = "libsql+ws://localhost:6969"
  exclude = ["_litestream*"]
}

env "turso" {
  url     = "libsql+wss://${getenv("TURSO_DB_PRL")}?authToken=${var.token}"
  exclude = ["_litestream*"]
  diff {
    skip {
      drop_schema = !var.destructive
      drop_table  = !var.destructive
    }
  }
}

my atlas apply command
atlas schema apply --env=dev --var destructive=true --to file://schema.hcl

My guess is that connecting to turso using a configuraton file has been left out and still only accepts wss:// scheme?
https://atlasgo.io/guides/sqlite/turso (this doc only mentions libsql+wss://)

Hello, I ran your script without any issue on the canary version of atlas-cli. You can install that version using this script
curl -sSf https://atlasgo.sh | sh

Please try again and let me know if the issue still existing. Also, we have a guide for turso here: https://atlasgo.io/guides/sqlite/turso

Ok I figured out where its going wrong... My nix flake for atlas v0.15.0  is failing to build so I was probably using v0.14.1 as fallback. Weird how it would show me the version 0.15.0 on atlas version though.
While 0.14.1 is built just fine, 0.15.0 fails to build.

I think the problem is introduced with this change:
cmd/internal/sqlparse/pgparse: move to pgquery (https://github.com/ariga/atlas/pull/2143)
Maybe this is relevant:
https://discourse.nixos.org/t/trouble-packaging-sqlc/18261/9
My undertanding of go and nix isn't deep enough for me to be certain though. Will continue to explore this more.

The issue has been resolved with this PR:
https://github.com/NixOS/nixpkgs/pull/267409


[Bug Report] - Argos CI Screenshots failing at run yarn playwrite test step

Describe the bug.
Run yarn playwright test
yarn run v1.22.19
$ /home/runner/work/uptane.github.io/uptane.github.io/node_modules/.bin/playwright test
Error: ENOENT: no such file or directory, open './screenshot.css'
at argos/screenshot.spec.js:10
8 | const sitemapPath = './build/sitemap.xml';
9 | const stylesheetPath = './screenshot.css';

10 | const stylesheet = fs.readFileSync(stylesheetPath).toString();
|                       ^
11 |
12 | // Wait for hydration, requires Docusaurus v2.4.3+
13 | // Docusaurus adds a  once hydrated

at Object.<anonymous> (/home/runner/work/uptane.github.io/uptane.github.io/argos/screenshot.spec.js:[10](https://github.com/uptane/uptane.github.io/actions/runs/6695724126/job/18191913819#step:7:11):23)

Error: No tests found
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
Describe the steps to reproduce the behavior.
Happens every time Argos CI Screenshots CI workflow is run.
Expected behavior.
It should run without issue.
Screenshots.
No response
Operating environment
Ubuntu
What browsers are you seeing the problem on?
No response
Additional context
No response

@devilkiller-ag the workflow can't seem to find screenshot.css. Perhaps a path issue?

Hi @hexsecs, I will check this out.

Hi @hexsecs I solved this error. However, I noticed that for argos to perform screenshot test we need build of website. So its failing locally giving this error:
$yarn playwright test
yarn run v1.22.19
$ /home/devilkillerag/Documents/OpenSource/uptane-website/node_modules/.bin/playwright test
Error: ENOENT: no such file or directory, open './build/sitemap.xml'

   at argos/utils.js:7

   5 | // Docusaurus generates a build/sitemap.xml file for you!
   6 | export function extractSitemapPathnames(sitemapPath) {
>  7 |     const sitemap = fs.readFileSync(sitemapPath).toString();
     |                        ^
   8 |     const $ = cheerio.load(sitemap, { xmlMode: true });
   9 |     const urls = [];
  10 |     $('loc').each(function handleLoc() {

    at extractSitemapPathnames (/home/devilkillerag/Documents/OpenSource/uptane-website/argos/utils.js:7:24)
    at /home/devilkillerag/Documents/OpenSource/uptane-website/argos/screenshot.spec.js:30:46
    at Object.<anonymous> (/home/devilkillerag/Documents/OpenSource/uptane-website/argos/screenshot.spec.js:29:6)
Error: No tests found

error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

If you are doing it locally you can call a yarn build first.

To manually test the steps locally I ran...
yarn install
yarn playwright install
yarn playwright test

I was able to get it work successfully pass.

@devilkiller-ag can you fix the path and submit a PR. For me, I got it working using the path ./argos/screenshot.css

@devilkiller-ag I think your other PR commit trashed the yarn.lock file also. That might be part of the issue. So actually the steps should be...
rm yarn.lock
yarn install
yarn playwright install
yarn playwright test

Yeah I resolved all issues. Before pushing I want to confirm that, do I need to put /screenshot created by running the teat to .gitignore file?


Flutter needs to integrate with Mozart Views to handle hit testing properly

@abarth
@Hixie

cc @chinmaygarde @abarth @jason-simmons Any of you able to look at this in the near future?

Aye. I am working on hit testing in the composer. Will follow up with @apwilson in person regarding the specifics.

@apwilson was asking about this bug today.  I assume @chinmaygarde is still on this?

I'm going to assume @chinmaygarde is on this for now, please un-assign if not.

cc @alanlgardner
I assume this is requesting that we provide a way to take an Offset in global coordinates from the top-left of our view, and return a list and a boolean, the list being the child views that we would consider hit at that point before us, and the boolean being whether we would consider ourselves to be hit at that point if none of those child views consider themselves to be hit.
Is that accurate?

@chinmaygarde, let me know if you'd any help with this one, especially on the framework side (I am less sure what needs doing on the engine side).

@chinmaygarde Is this what your big patch is doing?

This should be done already.

Cool, ok. Closing for now, please re-open if we haven't addressed the problem to your satisfaction!


Wrong URI or Link is dead at guarded-fabric-install-trusted-tpm-root-certificates.md

https://tpmsec.microsoft.com/OnPremisesDHA/TrustedTPM.cab redirect to https://microsoft.com/en-us

looks like it is fixed and points to https://go.microsoft.com/fwlink/?linkid=2097925.


[MACOS] No Spandoc settings in menu

This is a promising package, at least I need it a lot. Now when choose pallet I got your original path
C:/Users/robin/AppData/Roaming/cabal/bin/pandoc.exe and cannot find the settings to change this?

After the rewrite, I've forgotten to edit the Main.sublime-menu. Now you can go Preferences->Packages Settings->Spandoc and copy the default settings (which now do not contain my path anymore) to your user settings.
Notice however, that I've not tested my package on Mac, only on Linux (and Windows). Altough it should work with Mac.


Preview is Stretched

Thanks for this amazing library. I successfully implemented this in my project. But the preview is vertically stretched. I really need your help.
I am using Vivo 1814 for testing and it happens in all devices.

Did you try OpenGL example with keepaspectratio enabled?
https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/blob/master/app/src/main/res/layout/activity_open_gl.xml#L12

You are awesome, Thanks for the reply. I thank you from my whole complete heart. Now it's working, by mistake, I replaced the OpenGL view with SurfaceView of my Own.


add cc-rs no-defaults workaround to all configs

@SergioGasquez sorry for the ping can we get that out before you are on vacation?


Allow Karpenter SQS Queue Name to be set

Description
What problem are you trying to solve?
Say that you have 2 or more clusters in the same region and you want to avoid karpenter to use the same sqs queue , atm I personally didn't get how to set a different name than "karpenter" to the SQS queue so I'd like to be able to tell karpenter to use "my-queue" instead of the default one.
How important is this feature to you?
May become important in the near feature

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

nevermind, I found finally the
"aws.interruptionQueueName": "karpenter-whatever"


uORF name or definition change

We are looking at adding a regulatory_class for uORF or regulatory_uORF to the INSDC specifications, and are trying to sync up with SO.
SO term: uORF (SO:0002027)
The current term has the less specific name "uORF", but is defined as "plays a role in translational regulation"
Should the SO term be renamed as "regulatory_uORF" (which is currently a synonym), or should the definition be revised to be less specific?
I don't know if existing usage of SO:0002027 is constrained to uORFs with demonstrated regulatory uORF function. I suspect not. It may be cleanest to change the definition of SO:0002027 to be less specific, and add a child regulatory_uORF term that is more specific.

Hi SO -- have you had a chance to look at this? We need to decide on which term and definition to use for INSDC. Thanks!


The diagram in INTEGRATING.md is impossible.

Bug Description
The diagram in INTEGRATING.md is impossible. The relations don't work that way.
To Reproduce

Environment

Relevant log output
-

Additional context
No response

issue is the lower lxd deployment box is incorrect: you can't put scrape config between grafana-agent (machine charm) and prometheus, because it's a prometheus_scrape.


internalWebError: context deadline exceeded

error message seen at: https://ipfs.io/docs/commands/
via https://ipfs.io/docs/api

https://ipfs.io/docs/api page, linked from About, sends me to that error page

Should work now.
@jbenet:  I just republished the website. I guess the last version wasn't pinned properly?

thoughts @lgierth ?

I had pinned it...
Dunno, something timed out.

Can't replicate. Closing! Reopen if it pops up again.


Add functionality to browse modified files

This is my first ever contribution to an Open Source project, and I only know Lua from my NeoVim config.
If there is anything to improve or completely wrong, please let me know.
Since this PR is open, I worked with this branch, and it worked pretty reliable
Goals

When having a PR selected, browse the modified files for this PR with <C-f>
Add  a direct command to browser modified files for the current PR the user has checked out

Hi @nikbrunner
Thanks for your contribution.
The file browse feature worked fine in my tests.
I've merged this PR through the https://github.com/nvim-telescope/telescope-github.nvim/pull/34
Feel free to reopen this PR/create a new one.

@sbulav
Thank you very much!
I maybe want to refactor some code of mine, but I will open a new PR for it, when I do


Bellman Ford Algorithm in C

#496

Sir, I have made the required changes.

Add some comments to explain your core logic.

Sir I have made the changes yesterday itself.

Could you just add the explanation of the Bellman Ford Algorithm at the top of the file, make the explanation as if a novice person might get an idea, you are free to add some links as well.


⚠ Jenkins Plugins Artifactory has degraded performance

In c49c8ac, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1185 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 1d37db0 after 14 minutes.


Added a logo

Feel free to leave feedback, I'm pretty happy with it but I can change it around :smile:

It was my pleasure :)
I really liked the project so thought I return something :smile:

I named you in the README.md, hope you are okay with that :)

That's awesome :grinning: thank you!


Warning-only mode for ErrorHandlers

What if the ErrorHandler class to allowed for "warnings" instead of raising the error and applying the correction...? I'd have a fix_error=True attribute that is used in the .check() method. and If fix_error=False, I simply print a warning & also add that warning to simmate_corrections.csv

Elsewhere, Simmate avoids modification of settings so that a table only contains consistent results (see our tutorial discussion). I think a "warning-only" mode is inconsistent with this strategy -- as it would allow erroneous data into the results table. I've therefore decided against adding this feature and am closing the issue.


with vuetify 2.0.3 throwing template error

 ERROR  Could not compile template<EMAIL_ADDRESS>Cannot resolve "vuetify/src/stylus/main.styl" from "C:\Kiran\mygit\prod\chibu\client2\tktest\vuetify\src\stylus\main.styl"

  at Promise.all.templateFiles.map (node_modules\@nuxt\builder\dist\builder.js:5945:17)

It was my issue .
Some how there were multiple create-nuxt-app installed on my machine . I ran the below command and it cleaned the stuff .
npm uninstall -g create-nuxt-app
Now it is working just perfect with npm . sorry for the wrong call


[False Positive] Dropbox : dl.dropboxusercontent.com

Hello,
The domain used by dropbox to provide file download has been blocked:
dl.dropboxusercontent.com
I've reported it to the Steven Black blocklist that had your blocklist as feed:
https://github.com/StevenBlack/hosts/issues/1868
You might want to rework your url sanitization to domain, because for some service like Imgur, Dropbox, etc ... blocking the domain isn't a good idea when working from the URL.

Domain added to the whitelist.
URLs are malicious, so them will be maintained in the latesturls.txt file.
New lists will be published in minutes.
Thanks for your advice


How is the mini-batch loss/error different from training or test loss/error?

As far as I know, it is uncommon to describe evaluation metrics in terms of mini-batch

The training loss and error are computed at the end of each epoch without any network update similar to validation ones. However, the mini-batch loss and error are accumulated all the iterations while continuously updating the model.


How to get a list of presets/programs in soundfont

I don't think you've implemented the functions to do so. At least, your code doesn't contain any calls to _fluid_sfont_get_preset or _fluid_synth_get_program.

I don't think you've implemented the functions to do so

Yes, currently not implemented.
I think it's useful if SoundFont APIs are included in this library, so I'll implement in next release.

I added Synthesizer.prototype.getSFontObject in commit 5a8c336d.
For this commit, you can retrieve Soundfont presets as follows:
    const sfontId = await synth.loadSFont(binSFont);
    const sfont = synth.getSFontObject(sfontId);
    const allPresets = [...sfont.getPresetIterable()]; // each item of allPresets will be `Preset` instance
    // for retrieving preset names:
    const allPresetNames = allPresets.map((preset) => preset.getName());

getSFontObject doesn't work with AudioWorkletNodeSynthesizer

Sorry I forgot AudioWorkletNodeSynthesizer...
I commited 11ee6b4b for AudioWorkletNodeSynthesizer.

I released as v1.8.0.

Closing this issue. Feel free to reopen if some problems still persist.


prog_std.txt is missing

When running the script as such:
sudo ./X11rdp-o-matic.sh --branch master --nocpuoptimize

It seemed to run fine until the Building xrdp source and packaging stage:
...

make[2]: Leaving directory '/<my-dir>/X11RDP-o-Matic/work/xrdp-0.9.1+master'
make[1]: Leaving directory '/<my-dir>/X11RDP-o-Matic/work/xrdp-0.9.1+master'
   dh_installdocs
cp: cannot stat 'prog_std.txt': No such file or directory
dh_installdocs: cp --reflink=auto -a prog_std.txt debian/xrdp/usr/share/doc/xrdp returned exit c
debian/rules:18: recipe for target 'binary' failed
make: *** [binary] Error 2
dpkg-buildpackage: error: fakeroot debian/rules binary gave error exit status 2

I then also got this error, but my assumption is that it's only because of the above:
mv: cannot stat 'xrdp*.deb': No such file or directory

Same issue ubuntu 16.04

I started work on this issue.

I'm seeing this on ubuntu 16.04 too. Is there a workaround?

There's no workaround. Give me a time.

I've got the same error:
cp: cannot stat 'prog_std.txt': No such file or directory
installing on Ubuntu 16.04 using
sudo ./X11rdp-o-matic.sh --branch devel --justdoit
I downloaded a fresh X11rdp-o-matic to get rid of temp files, then I've edited the file debian\docs and deleted the line that references prod_std.txt, and it worked.

@kofalt @aleks-nl5 @JamieR007 @Peluko
I'm sorry to have kept you waiting so long.
I've succeeded to finish the build. Try this. The branch is not tested at all if the installed xrdp properly works. Please note this is call for ealrly testers. If this works fine, I'll merge this.
https://github.com/metalefty/X11RDP-o-Matic/tree/work
Run o-Matic with this option. Building master branch is not supported at this testing.
./X11rdp-o-matic.sh --branch devel --nox11rdp

Now built xrdp is not correctly working. I'll have a look.
Please test only whether the build finishes successfully.

There's a few more things to do.

Sounds good. I didn't reply, but I'm happy to wait until you're ready for me to try things out.


[Suggestion?] Chat transparency fading vertically (bottom to top)

Hi again,
I have a suggestion, but aware people might not agree. :P
Just like the chat fades from left to right in transparency, becoming completely transparent at the end of the chat box, I think it would look better if this was also matched vertically (from bottom being the value you choose, to the top being completely transparent).
Currently, I think the chat box cuts off abruptly under the chat tabs and would look better smoothed out. But I could be compltely wrong, just imaging it in my head!

Ik you closed it, but yeah, I don't like it either. But it's kinda impossible to add the fading because you can't apply alpha proper alpha masks to text. And resizing the chat frame dynamically isn't an option either.
However, I'm experimenting with some possible design changes to make it look less jarring, like, right now the main issue is that it's "cut off" by nothing:

But what if I added something at the top? 🤔
I'll reopen it, so people don't ask the same question again and again.

Amazing to hear. I just didn't want to be annoying as I opened another request, but I love this addon so much.
The standard wow chat is such a terrible part of the UI imo, so this is like my only hope :D

Yeah, it annoys me as well. So dw 🤗
Sadly, due to rl issues I haven't been playing wow much. Like, I unlocked daily quests on my main only a week ago and I hit level 70 in early December! So yeah... I wish I could do more and quicker, but 😓

No stress, all is good! Take care of yourself!


Thoughts on switching from CodeMirror to Monaco Editor?

https://monaco-react.surenatoyan.com/
I've been using the above in a project recently the an intellisense is really fantastic. Any thoughts on swapping out code mirror for monaco editor?

Monaco looks good, also considering its integration with React. Will need to test it. Thanks @armaandh


Vertex colors export option missing in 4.1

Describe the bug
The option to export vertex colors is missing from the export window. Re-importing the output file in Blender confirms that no vertex color data was exported. In previous versions (4.0.x), the vertex color checkbox is there and works as expected.
To Reproduce
Steps to reproduce the behavior:

Open attached file, then go to File > Export > glTF 2.0
Under Data > Mesh, the Vertex Color checkbox is missing.
Export to glb, then open new blender file.
Re-import the exported file into blender, or import provided glb file
In vertex paint mode, vertex colors are missing.

Expected behavior
Vertex colors are exported properly in resulting .glb file.
.blend file/ .gltf (mandatory)
vertex-color-export-test.zip
Version

OS: Windows-10-10.0.19045-SP0 64 Bits
Blender Version 4.1.0

Hello,
The way Vertex Color has changed between 4.0 and 4.1.
To be more aligned with Blender viewport, we now export VC only when the VC is used in the node graph.
Just saw that the Blender documentation was not updated
Note that option to export multiple VC without exporting additonal VC as custom properties is under discussion : #2195

Closing, this ticket, as there is no issue here.
For your export :

or

hi, is there any particular reason for this change? I only use vertex colors purely as data, and I don't want it affecting the color output in the material. If I have to use it in the node graph, how can I set it to not affect base color?

If you want to use purely data without any impact on colors, you should use custom attributes.
VC are used to change colors, not to store customs data, where glTF already have this data management.
You should enable custom attributes export, and VC colors will export as custom attributes

"VC are used to change colors, not to store customs data"
almost every 3d game ever made would disagree


[Question]: For the CrewAI example, which Wolfram API service is the correct one?

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
Re: https://github.com/run-llama/llama_index/blob/main/docs/docs/examples/cookbooks/crewai_llamaindex.ipynb
Tried the "full service" and "simple answers" ones and getting "invalid appID".  TIA.

Using the "full answers" one works now.  Must have taken a few minutes to register with the service.


Cannot install on Alpine

Hi! I'm trying to run a NodeJS project that uses sleep, but I can't:
module.js:597
  return process.dlopen(module, path._makeLong(filename));
                 ^

Error: Error loading shared library /x/node_modules/sleep/build/Release/node_sleep.node: Exec format error
    at Error (native)
    at Object.Module._extensions..node (module.js:597:18)
    at Module.load (module.js:487:32)
    at tryModuleLoad (module.js:446:12)
    at Function.Module._load (module.js:438:3)
    at Module.require (module.js:497:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/x/node_modules/sleep/index.js:1:75)

I've tried other solutions (google, etc). How can I force to use the fallback?

How did you install it. Can you give me the output of npm install --verbose sleep

@erikdubbelboer I don't remember what I did, but I solve that. I will be closing this ticket. Thank you!


The use-parent-property linter rule for virtualNetwork does not work as expected

Bicep version
Bicep CLI version 0.15.31 (3ba6e06a8d)
Describe the bug
The use-parent-property linter rule for virtualNetwork does not work as expected and produces an error
To Reproduce
The use-parent-property linter rule for virtualNetwork does not work as expected

When you perform the quick-fix, it produces an error

See the sample described in #9024 for a workaround using any()


Specify location for serverCache.json, create option for serverCache update frequency, allow for local server list

Currently serverCache.json is saved to wherever you run the program from.  This could be the location of ipwatch.py, it could be the home directory, or somewhere else.  This should be normalized and probably made an option in the config.
Currently the update frequency of serverCache.json is hard coded to 90 days.  This should probably be made as a config option and should likely default to a more frequent update rate.
There should be an option to use a local server list instead of forcing the SW to use the list from github (and/or overwrite a custom user list when it is updated).

:+1:
When I enter the directory of ipwatch and execute it continuously, it works well, using serverCache.json without need of github.
But when it triggers by crontab, due to it cannot find the location of serverCache.json, it tries to download from github every time.
00 * * * * python3 /opt/ipwatch/ipwatch.py /opt/ipwatch/config.txt

Yeah I was just noticing the other day that tasks run by a crontab execute "from" the root user home folder at /root.  This is where I found my serverCache.json file.  As I mentioned above, I think I'll add a serverCache.json save location to the config file soon.
However, I'm surprised to hear you say it "tries to download from github every time."  I would have expected it to be in your /root folder (and then not have to download after that) as well.


Adding explanations to default guardrails

Adding explanations to the data property of default guardrail plugins
We're adding a data.explanation property to default guardrails to make it easier to identify why they succeeded or failed.

Changing this to draft. Please update the PR once its ready

@VisargD all tests for default plugins are passing. You'll need to check the portkey plugins with credentials. I could not run those tests


fix(link): update link to IBM cloud in Jumbotron

The URL should be
https://console.bluemix.net/registration/trial?target=%2Fcatalog%2Fservices%2Fnatural-language-classifier%3FhideTours%3Dtrue%26cm_mmc%3DOSocial_Tumblr-_-Watson%2BCore_Watson%2BCore%2B-%2BPlatform-_-WW_WW-_-wdc-ref%26cm_mmc%3DOSocial_Tumblr-_-Watson%2BCore_Watson%2BCore%2B-%2BPlatform-_-WW_WW-_-wdc-ref%26cm_mmca1%3D000000OF%26cm_mmca2%3D10000409%25

Was checking out the link: I get a 500 error for it. It works if I remove everything after hiding the trial: https://console.bluemix.net/registration/trial?target=%2Fcatalog%2Fservices%2Fnatural-language-classifier%3FhideTours%3Dtrue


SingleWebPartAppPage navigation

Category

[x ] Question
[ ] Typo
[ ] Bug
[ ] Additional article idea

When we have multiple SingleWebPartAppPage pages, navigation using links in the Quick Launch don't work as expected.  On first click of a link, the browser navigates as normal.  However, to then move to another page, the user has to click the link twice, and the navigation only occurs on the second click.
It looks like on the first click, the browser makes a call for the page with ?as=json added.
I saw similar issues before SingleWebPartAppPage came out in preview when we used to use Home rather than Article layouts for our app pages to remove the header.
Is there any guide to how SharePoint navigation works or how to make SingleWebPartAppPage work properly with Quick Launch?

Maybe these links will help for you:
ng-marcusNavigating from a page to another page causes SPFX web part on secondary page to not load #2398
Navigating from a page to another page causes application state issues #1700

Thanks,
#2398 looks the same issue, but it seems the cause of that was already fixed, unless that rollback was on a per-tenant basis?

As Vesa mentioned, this is fixed in the GA bits.  I am closing this now, if after the GA of 1.8 it still has issues please respond (I validated it works though).

I built with 1.8 and see no change in behaviour.    We also see the issue reported here:
https://github.com/SharePoint/sp-dev-docs/issues/3566

As Vesa mentioned, this is fixed in the GA bits. I am closing this now, if after the GA of 1.8 it still has issues please respond (I validated it works though).

Hello, I'm seeing no change in behaviour unfortunately when building with 1.8

@ng-marcus - Sorry that looks like the patching of our bits has not rolled out yet.  By next Monday that fix should be visible.  If you would like I can verify that is the issue if you send me a fiddler trace to grahamc at my Microsoft account.

Thanks for the update.
I think the problem is generic, as I made two SingleWebPartAppPages with just a code preview element, and I see the same issue.   Happy to hear the fix is still coming, so I'll give it some time.  I'll try and get a fiddler trace in the meantime.  Many thanks!

I opened #3566 related to this issue. @ng-marcus referred me to this thread. I have completed upgrading to SharePoint framework 1.8 and build package solution. The issue reported isn't resolved yet. I will look forward for the fix coming by next Monday.

Very happy to report this issue is now resolved for me.   Single clicks of the quick links lead to an immediate navigation to the target page.

@ng-marcus great to see this issue is resolved. What has changed since last time to resolve it? Have you done anything different from your side recently? Thanks.

We rebuilt our app with spfx 1.8.  I think you need both the server side and client site update for this to be fixed.

@ng-marcus how do you update server side to spfx 1.8? I have followed release notes to 1.8, https://github.com/SharePoint/sp-dev-docs/wiki/SharePoint-Framework-v1.8-release-notes.

The only thing you should need to do is rebuild your app with the 1.8 version of the framework and deploy it (we use the AppCatalog).
The server update is being taken care of by Microsoft, and should have hit their servers by today (Monday).  We saw our sites update on Saturday night.

Thanks for confirming it is fixed.
@beyasu - you actually don't even have to upgrade to 1.8.  The fix will just roll out and you will automatically get it.  I am closing this issue as it is now confirmed fixed.  However, if you are still seeing this issue please send a fiddler trace to grahamc at my microsoft.com address and I will confirm whether the issue is that you are still not upgraded or if there is another issue.

@mcmynn83 I just checked our sites and this issue is resolved from from our side too. Thanks!!

Excellent glad to hear the issue is fixed.

We're having a similar issue currently. Using a SPFx 1.7.0 webpart, when we force the PageLayoutType to "SingleWebPartAppPage" (doc), the onDispose() method is never called resulting in weird behaviour when switching between pages. @VesaJuvonen @GrahamMcMynn
Perhaps SingleWebPartAppPages are not supported for 1.7.0 webparts? Do we need to upgrade to 1.8.x?
Kind regards


Nokogiri doesn't install on windows

Recent security update to gem executable has broken nokogiri fat gem on windows (tested on windows 7, 64bit, 2.5.0 rubyinstaller). Error is:
ERROR:  While executing gem ... (Gem::Package::PathError)
	installing into parent path bin/nokogiri of C:/dev/Ruby25-x64/lib/ruby/gems/2.5.0/gems/nokogiri-1.8.2-x64-mingw
32 is not allowed

I suspect this https://www.ruby-lang.org/en/news/2018/02/17/multiple-vulnerabilities-in-rubygems/ is relevant because it installs fine without that update.

I can not reproduce this so far. Both fat binary gem and source gem install fine for me.
When is the error raised? If it's while gem install, could you please re-run
gem env
gem inst nokogiri --verbose --backtrace

and post the output?

                                                                                Just to make sure, did you first upgrade ruby gems usinggem update --system?                                                                                From: Lars KanisSent: Saturday, February 24, 2018 23:36To: sparklemotion/nokogiriReply To: sparklemotion/nokogiriCc: W.; AuthorSubject: Re: [sparklemotion/nokogiri] Nokogiri doesn't install on windows (#1726)I can not reproduce this so far. Both fat binary gem and source gem install fine for me.

When is the error raised? If it's while gem install, could you please re-run
gem env
gem inst nokogiri --verbose --backtrace
and post the output?
—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or mute the thread.
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/sparklemotion/nokogiri","title":"sparklemotion/nokogiri","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/sparklemotion/nokogiri"}},"updates":{"snippets":[{"icon":"PERSON","message":"@larskanis in #1726: I can not reproduce this so far. Both fat binary gem and source gem install fine for me.\r\nWhen is the error raised? If it's while gem install, could you please re-run\r\n\r\ngem env\r\ngem inst nokogiri --verbose --backtrace\r\n\r\nand post the output?\r\n"}],"action":{"name":"View Issue","url":"https://github.com/sparklemotion/nokogiri/issues/1726#issuecomment-368265722"}}}

@graywolf Of course, I upgraded to rubygems-2.7.6 before. However I noticed a very similar issue on appveyor, which I can not reproduce locally too.

Tomorrow when I get back to my windows machine I'll provide the requested information :)

Sorry for wasting your time, I figured it out and it's not nokogiri issue. Will make bug report with ruby gems directly when I have some time.

I've done a little bit of printf debugging on Appveyor and found the root cause. It happens if you use rubygems-2.7.6 and set a GEM_HOME with different upper/lower case characters than the directory has on the filesystem. Possibly there are even more settings which result in this error, like rubygems config files, etc.
I'll file a bug report on rubygems.

@graywolf @larskanis thanks to both of you for following up!

@larskanis Just for when you are reporting it: For me it's issue with TOPDIR in general, not with GEM_HOME. RbConfig::TOPDIR points to C:/dev/Ruby25-x64 while it should be C:/Dev/Ruby25-x64. So when making the bugreport, could you mention that you cannot really rely on __FILE__ in /lib/ruby/2.5.0/x64-mingw32/RbConfig.rb having correct casing of folders.


[DO NOT MERGE][AUTOREST V3] generated storage multiapi with new autorest v3

Can one of the admins verify this patch?


libc/execinfo: extract a common backtrace format function

Summary
Extract a common method to format backtrace.
Impact
No.
Testing
boards/arm64/qemu/qemu-armv8a/configs/nsh with related options enabled.

please add a commit message

please add a commit message

Commit message updated.


Use same repo name, Find-Modules should not use *

use the same repository name throughout.  In step 5, Find-Module uses "PowershellAzureDevopsServices" and Install-Module uses "PowershellVSTS".  They should match.
The Find-Module command should not include the *.  It should be "Find-Module -Repository PowershellAzureDevopsServices -Credential $credsAzureDevopsServices"

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

ID: 35b3cd51-b365-433b-9149-697582d64e88
Version Independent ID: 1cb1f4a3-5928-223d-592f-f3462fef4e4c
Content: Use Azure Artifacts as a private PowerShell repository
Content Source: docs/artifacts/tutorials/private-powershell-library.md
Product: devops
GitHub Login: @elbatk
Microsoft Alias: elbatk

Thank you so much @pspill , I'm updating these now to help future users.


Is there an example of Chainlit working with Microsoft Guidance

Microsoft Guidance seem to be a powerful prompt engineering tool. Is there a working example of how chainlit works with this.
I came up with an example, but not sure if this is a best practice.
@cl.on_chat_start
def start_chat():
    guidance.llm = guidance.llms.OpenAI(...)

@cl.on_message
async def main(message: str):
    program = guidance('''
        {{#system~}}
        You are a helpful assistant
        {{~/system}}

        {{~#geneach 'conversation' stop=False}}
        {{#user~}}
        {{set 'this.user_text' (await 'user_text')  hidden=False}}
        {{~/user}}

        {{#assistant~}}
        {{gen 'this.ai_text' temperature=0 max_tokens=300}}
        {{~/assistant}}
        {{~/geneach}}''')

    program = program(user_text =message)

    print(program['conversation'])

    await cl.Message(content=str(program)).send()

This works great. But I get below prompt in my frontend. Not sure if there is a way to format this.
Chatbot

09:37:33 PM

<|im_start|>system
You are a helpful assistant<|im_end|>
<|im_start|>user
hi there<|im_end|>

    <|im_start|>assistant

Any help will be appreciated.

I was not familiar with this library. I played a bit with it and here is how it can integrate with Chainlit at first glance:
import chainlit as cl
import guidance
from guidance._program import Log

guidance.llm = guidance.llms.OpenAI(model="gpt-3.5-turbo")

class ChainlitLog(Log):
    def append(self, entry):
        super().append(entry)
        print(entry)
        is_end = entry["type"] == "end"
        is_assistant = entry["name"] == "assistant"
        if is_end and is_assistant:
            cl.run_sync(cl.Message(content=entry["new_prefix"]).send())

@cl.on_message
async def main(message: str):
    program = guidance(
        """
        {{#system~}}
        You are a helpful assistant
        {{~/system}}

        {{~#geneach 'conversation' stop=False}}
        {{#user~}}
        {{set 'this.user_text' (await 'user_text')  hidden=False}}
        {{~/user}}

        {{#assistant~}}
        {{gen 'this.ai_text' temperature=0 max_tokens=300}}
        {{~/assistant}}
        {{~/geneach}}""",
    )

    program(user_text=message, log=ChainlitLog())

This is very minimal and we have to explore what exactly is available in the entry dict. Also some string cleaning will be necessary.


Wrong rate_limit_per_user interval for Channel

For some unknown reason I was banned from the development Discord server, so I come to make the request directly here ...
The interval does not correspond at all to the official documentation, preventing to set a cooldown of more than 2 minutes.
https://github.com/shardlab/discordrb/blob/dc27fe18463da3ccfd0f0266030aa7ad51b2c2b9/lib/discordrb/data/channel.rb#L350
The value should be 21600 instead of 120

Thanks in advance for the modification!

More than happy to update this
@Dakurei can you join our server and give me your discord ID so I can look into that for you? https://discord.gg/Mz2hm8Vw

Nevermind, I found your case and unbanned you from the Discord API server 🙂

Thanks a lot !

I made the modification, and I took the occasion to fix the test that was failing.
Normally it should be good, I'll let you check and merge it when you have the opportunity 👍


Replace an undocumented example by a cookbook page

Add a single page in the cookbook, replacing a python/octave/ ... example.
The patch should contain 3 changes

added .rst file
added .sg file
remove .py, .m, ... files

See https://github.com/shogun-toolbox/docs/blob/master/EXAMPLES.md and https://github.com/shogun-toolbox/docs/blob/master/DEVELOPING.md for instructions.
NOTE: There are many many of those, so this needs many people solving small pieces

Ok, I've given it a shot for the Gaussian Naive Bayes example. I'm new to Shogun (and ML in general), so I would really appreciate it if someone could give it a quick look and let me know if everything looks ok.
https://github.com/tbskm/gaussianNaiveBayes-cookbook

Hi there,
great, these are exactly the two files we need. Please send a Pull request against the cookbook feature branch (link above) rather than posting the files here. I will review it in there then.
A note for the others -- there are many more examples to go

@karlnapf I tried to build the cookbook in doc/cookbook/build by doing cmake .. and make cookbook. I tried the same in shogun's build dir also. Here is the error gist.
cmake -DBUILD_META_EXAMPLES .. is also not working. What am I missing?

This is just the link checker failing, are you online?
Also pull the latest version, it works on my machine

Hmm, pretty sure I would have been online, but I'm using a stock Broadcom card that frequently acts up with Linux, so my connection probably went down at some point.
Btw I've been busy with some school assignments, but I should be able to remove those unnecessary bits from the cookbook page either today or tomorrow.
Thanks!
_____________________________

From: Heiko Strathmann<EMAIL_ADDRESS>Sent: Thursday, March 10, 2016 7:20 AM
Subject: Re: [shogun] Replace an undocumented example by a cookbook page (#3000)
To: shogun-toolbox/shogun<EMAIL_ADDRESS>Cc: Trevor Ballard<EMAIL_ADDRESS>This is just the link checker failing, are you online?
Also pull the latest version, it works on my machine
—
Reply to this email directly or view it on GitHub.

@karlnapf It works now after pulling the latest version. Although cmake -DBUILD_META_EXAMPLES .. is still not working. The cookbook looks pretty neat.

@tbskm I like the gaussian naive bayes example! Will merge soon.

@sanuj lets not discuss this here ... irc maybe?

Hello, I'm working on replacing "converter_multidimensionalscaling_modular.py" with a meta example and a cookbook page.
I have created a "converter" subdirectory under "examples/meta/src" and am thinking of adding a "Feature Transformations" (inspired from "converter") field in index.rst. Truth is that MDS is a general technique that can be applied for visualization, dimensionality reduction or  feature extraction, so I'm skeptical about the correct name, do you have any suggestions?
I also have a problem with this line of code
return numpy.linalg.norm(distance_matrix_after-distance_matrix_before)/numpy.linalg.norm(distance_matrix_before) < 1e-6
How should this appear in the .sg file? Building as
bool output = ((numpy.linalg.norm(distance_matrix_after-distance_matrix_before) / numpy.linalg.norm(distance_matrix_before)) < 1e-6) shows that the math operators are not used correctly. Any resources on this? To my knowledge no existing example uses a math operator other than =.

All good. I commented that we will not have this norm comparison in the output. That is unit testing material and the meta examples just store the results, so just remove the line

Whoops. Re-opening as there are lots of python example still.
No need for cookbook rst pages for the remaining distance examples. A meta example is fine.

I would like to give this a shot. Any suggestions on which example I should choose first?


Some problems about support for Volta GPU architecture(V100 and CUDA9/CUDNNV7).

1 Not support all features in Volta GPU.
I found Mxnet used SgemmEX API when the I/O is FP16 type. The input is FP16, and computing with FP32. So is cudnn_convolution.
2 How can we get 3.5x faster than Pascal
After I check in the real Hgemm API and fix the cudnn convolution compute type to FP16, the speed up to 2x faster than Pascal in VGG network. So which net mxnet team to train to achive 3.5x speedup?
3 Tensor mode
In convolution, mxnet enables tensor op only through set math mode, but in API documentation, using tensor op requires setting the corresponding forward-backward algorithm, such as CUDNN_CONVOLUTION_FWD_ALGO_IMPLICIT_PRECOMP_GEMM,  and the need for correct input and output data.
Environment

Hardware: V100-PCIE-16G
Software: Mxnet-0.12 CUDA-9.0 CUDNN-7
Network: VGG

You don't need (and in most cases you don't want) to set fp16 compute type to use TensorCore - it is doing internal accumulation in fp32, that is why it is exposed via functions like sgemmEx. Setting the cudnn convolution algo is done via the cudnnFind function (it is very beneficial to set the MXNET_CUDNN_AUTOTUNE_DEFAULT environment variable to 2).
You can look at the fp16 examples in example/image-classification. The typical benchmark that we perform for synthetic data is (I assume single GPU here)
python example/image-classification/train_imagenet.py --gpu 0 --batch-size 128 --num-epochs 1 --disp-batches 100 --network resnet-v1 --num-layers 50 --dtype float16 --benchmark 1
On Pascal you would need to set the batch size to 64 or 96 and dtype to float32.

@ptrendx
I think, Tensor op internal calculation method is FP16, not FP32. You can refer to the following document diagram.

It is either FP16 or FP32, and the difference in performance between those 2 is pretty negligible, but the accumulation precision is important for training.
I am from NVIDIA, by the way ;-).

@ptrendx
I got it.Thank you for your reply.
But, why you do not deal with the FP16, I found that in the absence of Tensor mode, in fact mxnet still use the input FP16 with calculation of FP32 to deal with. In the convolution OP only need to modify the computetype to FP16 .
I have test the tensor mode in cublas. Both Sgemm and Hgemm works well. And Hgemm get a bigger speedup than Sgemm.

@ptrendx
OK, I got 3.33x speedup between V100-tensor with P40 in resnet network.
Thank you for your assistant.


Support CYGWIN

This patch was contributed by Ken Brown (@kbrow1i) to properly support compiling the library on CYGWIN.
Should I perhaps switch to the dev branch? I read that after pushing the branch, I can still change it.
(It would probably be helpful to add a CI running on cygwin to properly test this, but I'm not sure how to use cygwin with Azure Pipelines. I see that some GitHub Actions exist to support cygwin tests.)

Thanks so much! I added this directly to dev/dev-slice as the branches have diverged a lot recently. Thanks.


allow to enter/activate vertices using double-click

A new graph-action should be added that is invoked on double click of a given vertex.

Implemented on master (v0.5.0).


Support uppercase and lowercase letters and other extensions

I'd like to be able to get license files for uppercase and lowercase letters, .md and other extensions as well.
For example, I want to target license files that exist in the following repositories.
https://github.com/josharian/intern
While the above repositories are indeed MIT-licensed, they are not covered by go-licenses.
Thank you :)

same https://github.com/google/go-licenses/pull/19

I think #19 would solve this, would you mind taking a look at it to confirm?

cc @wlynch the issue should be closed now

Thank you!


feat: supoort collapse unchanged regions

Types

[x] 🎉 New Features

Background or solution
preview:
https://github.com/opensumi/core/assets/9823838/3f93dc0a-fa7e-467a-ab60-1ae5cfbfb6d7
Changelog
support collapse unchanged regions on diff editor

/next


support for yaml config files

Would you consider supporting config files in yaml format ?
This would allow us to use comments in the config files as well as profit from achors/aliases

I don't think so, but we can improve things!
Having two configuration formats means more documentation and inconsistency between configurations.
I will take the issue in another way:

why did you need comments?
why did you need YAML specific features as anchors and alias?

I need concrete use cases to justify such significant work like that.
In conclusion, isn't YAML overkill?

Why comments ?
When your config file grows big, and has to be maintained by a team, it
would be very useful to add comments. Comments can justify why things are
done a certain way that might feel weird to newcomers. Give guidelines, etc.
Why yaml anchors/alias ?
For Dont Repeat Yourself. i ended up repeating the same tiles quite a lot
in my first config file since I have the same things to check on multiple
servers. Yaml anchors could have been handy.
Finally, a yaml document can be expressed in json. So support for this
format can be implemented as a transformation into json, and the generated
json can feed the existing implementation without any change
Le ven. 13 mars 2020 à 00:16, Alexandre Demode<EMAIL_ADDRESS>a
écrit :

I don't think so, but we can improve things!
Having two configuration formats means more documentation and
inconsistency between configurations.
I will take the issue in another way:

why did you need comments?
why did you need YAML specific features as anchors and alias?

I need concrete use cases to justify such significant work like that.
In conclusion, isn't YAML overkill?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/monitoror/monitoror/issues/267#issuecomment-598473315,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAKKPAYX6CJS27WEDT2GTDRHFUNHANCNFSM4LGWY75Q
.

As I said, implementation is not the question, and it's easy to do.
The cost is to maintain documentation of two formats.
Comments can be added to JSON if we want, I think. So the only remaining point is anchors.
We will take a look at that with @jsdidierlaurent

After a short discussion with @jsdidierlaurent, we will not add YAML support. The gain is low, YAML is for power users, and Monitoror is not only for them.
We will add a way to add comments to the JSON, more like notes than comments via a note field to stick to the JSON 1 standard.

Actually @Alex-D I recommend you take a look at viper which is an industry-standard tool for parsing and writing out config files of all formats, with no extra work to support many formats. I've used it extensively and it's a fantastic package :)

This is not a technical issue. We already use viper.
The issue is: we need to write and maintain documentation for JSON AND YAML... + Consider all beginners who will have problems with YAML.
For now, we did not have time/energy to put in that.
If you want to use YAML, feel free to parse it to JSON :)

Ah, documentation is the issue. Got it. Totally makes sense.


Find a free coverage bot and set it up

I dunno where to host pytest --cov, or I'd just set that up myself right now.

I have no idea how to vet Codecov, but it seems to be working for MycroftAI, and it's free for FOSS. Drops that nice little comment in the PR, no need to get coverage into an Action or store its output.
It does require access via a GitHub account with repo perms, so it would probably be best to set up some kind of devops GitHub account for the OVOS org, assuming the GitHub terms allow that. I know companies do it sometimes, but I don't know if you have to spend money to be allowed.
@NeonDaniel?

Iirc, GitHub allows bot accounts with some "be reasonable" caveats. I haven't looked at coverage, but actions generally don't require any special account and show up as a GitHub bot

The one in question shows up in repos as itself, but it wants repo perms in order to do its work (and edit status lines.) I think it would be better if things like that got in through an OpenVoiceOS account, rather than one of our accounts, both so that things keep working if someone leaves the org and so that we can cut off its head in the event of an emergency.

Iirc, GitHub allows bot accounts with some "be reasonable" caveats (okay for actions, other automation, etc.). I haven't looked at coverage, but actions generally don't require any special account and show up as a GitHub bot

Documentation referenced: https://docs.github.com/en/get-started/learning-about-github/types-of-github-accounts

User accounts are intended for humans, but you can give one to a robot, such as a continuous integration bot, if necessary.

God, I love these people. And I'm a GitLab expat.

Closed by #75


kat: tracing test for HTTP auth service

Description
This PR adds a test for verifying that an external authorization server receives the proper tracing
headers when Ambassador is configured with an HTTP AuthService.

Test failures seem to be unrelated to the changes.

Hmmmm, I'm a little more concerned -- we don't typically see TracingTest flakes. It works for you locally?

It passes locally, but I can take another look at.

What is failing locally for me now is this assert:
assert self.results[100].backend.response[0] == 'tracingtest-default'

@kflynn Tests are fixed. PTAL, thank you!


[Bug]Unhook PlantUml from the install step as it causes CI failures

Bug Description

Intermittent failure on getting the PlantUml.jar during CI builds.
PlantUml is not needed for CI tasks, this is likely caused by a prepare hook - stop this from happening (may need to alter out PlantUml fork)
npm ERR! code 1
npm ERR! path /home/runner/.npm/_cacache/tmp/git-cloneYjSjij
npm ERR! command failed
npm ERR! command sh -c node scripts/get-plantuml-jar.js
npm ERR! Downloading plantuml.jar version 1.2021.12
npm ERR! node:events:368
npm ERR!       throw er; // Unhandled 'error' event
npm ERR!       ^
npm ERR!
npm ERR! Error: socket hang up
npm ERR!     at connResetException (node:internal/errors:691:14)
npm ERR!     at TLSSocket.socketOnEnd (node:_http_client:471:23)
npm ERR!     at TLSSocket.emit (node:events:402:35)
npm ERR!     at endReadableNT (node:internal/streams/readable:1343:12)
npm ERR!     at processTicksAndRejections (node:internal/process/task_queues:83:21)
npm ERR! Emitted 'error' event on ClientRequest instance at:
npm ERR!     at TLSSocket.socketOnEnd (node:_http_client:471:9)
npm ERR!     at TLSSocket.emit (node:events:402:35)
npm ERR!     at endReadableNT (node:internal/streams/readable:1343:12)
npm ERR!     at processTicksAndRejections (node:internal/process/task_queues:83:21) {
npm ERR!   code: 'ECONNRESET'
npm ERR! }

Problem is caused by the Sourceforge download of the underling JAR. Unless we copy & host, we're not escaping this problem.
Removing PlantUML as a dependency is the way forward, especially as nobody is using it 😞


switch to jiff from chrono

This PR migrates uv's use of chrono to jiff.
I did most of this work a while back as one of my tests to ensure Jiff
could actually be used in a real world project. I decided to revive
this because I noticed that reqwest-retry dropped its Chrono dependency,
which is I believe the only other thing requiring Chrono in uv.
(Although, we use a fork of reqwest-middleware at present, and that
hasn't been updated to latest upstream yet. I wasn't quite sure of the
process we have for that.)
In course of doing this, I actually made two changes to uv:
First is that the lock file now writes an RFC 3339 timestamp for
exclude-newer. Previously, we were using Chrono's Display
implementation for this which is a non-standard but "human readable"
format. I think the right thing to do here is an RFC 3339 timestamp.
Second is that, in addition to an RFC 3339 timestamp, --exclude-newer
used to accept a "UTC date." But this PR changes it to a "local date."
That is, a date in the user's system configured time zone. I think
this makes more sense than a UTC date, but one alternative is to drop
support for a date and just rely on an RFC 3339 timestamp. The main
motivation here is that automatically assuming UTC is often somewhat
confusing, since just writing an unqualified date like 2024-08-19 is
often assumed to be interpreted relative to the writer's "local" time.

Since e change interpretation of exclude newer times this is "breaking", right?

Since e change interpretation of exclude newer times this is "breaking", right?

Yup.

This is also breaking because uv won't parse old lock files either, since the datetime formats won't match.

Let's just group it into 0.3.0, we might start merging things for that today? I'll let you know.

OK, with #6206 merged, Chrono is now removed from uv's dependency tree:
$ cargo tree -e normal | rg chrono
$

chrono does still appear in the lock file, but I think that's from
axoupdater->homedir->wmi->chrono.

Let's just group it into 0.3.0, we might start merging things for that today? I'll let you know.

SGTM.


rotate_iou_gpu_eval seems to be wrong

When I test the code with single point cloud, and fed its label as detections as follows:
gt_annos[0]['score'][:] = 1.0 get_official_eval_result(gt_annos, gt_annos, ...)
it shoudl return an square matrix with it's diagonal is 1. but in bev and 3d in utils/eval.py: calculate_iou_partly(), it doesn't output as expected.

@poodarchu @traveller59
hello all,  i am facing the same situation.
Though this is a old issue, may I know if you have fix it?
Looking forward to your reply.


Changing values inside components

I am relatively new to Vue and trying to customize the daypasn vuetify plugin.
Thanks to Slots I have been able to customize many of the aspects of the plugin. Slots are easy to understand. However what I still dont get quite well is how to change the values of parametes in the inner components. For instance, lets say that Im trying to hide or shoe  the edit button inside the Calendar Event Popover component, according to certain conditions in the specific event. Than button is not inside a Slot but it has a v-if condition based in the value of "allowEdit"
Heres the code at src/components/CalendarEventPopover.vue:
<v-btn
       v-if="allowEdit"
       color="secondary"
       small absolute bottom left fab icon
       @click="edit">
       <v-icon>edit</v-icon>
     </v-btn>

So to show it or hide it I need to modify the value of allowEdit.
How can I modify this value from a custom method at my App.vue component?
Also Id like to know if there is any "create" event that triggers when the poover is being created?

Better use this vuetify calendar

If you look at the source you'll see that is a computed value:
https://github.com/ClickerMonkey/dayspan-vuetify/blob/master/src/components/CalendarEventPopover.vue#L290
You can see in the code it depends on the following props:

allowEditOnReadOnly
readOnly

And also the global $dayspan.readOnly variable and the readonly variable that can exist on your event details object.
When it comes to specifying props you can always check out this file:
https://github.com/ClickerMonkey/dayspan-vuetify/blob/master/src/defaults.js#L125
This file lists all the components and their default prop values.
You can overwrite any of the default prop values when you initialize the plugin like so:
Vue.use( DaySpanVuetify, {
  // options is vue definition, the resulting reactive component is stored in components as this.$dayspan or Vue.$dayspan
  data: {
    defaults: {
      dsCalendarEventPopover: {
        allowEditOnReadOnly: true
      }
    }
  }
});

And I believe you could also do this.$dayspan.defaults.dsCalendarEventPopover. allowEditOnReadOnly = true if you really wanted to. It would need to be done before the componet is created.


Pilot: Rename some functions for better readability

And facilitate v1alpha3 transition. There is no functional change.
Signed-off-by: Shriram Rajagopalan<EMAIL_ADDRESS>

@rshriram: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
386bb63a72471f5ce712f257cca97dc1b5dd9557
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Jetson setup

What type of PR is this? (check all applicable)

[x] Refactor
[x] Feature
[ ] Bug Fix
[ ] Optimisation

[required] Description
This replaces any Jetson Xavier NX environment and infrastructure settings to work with our new Jetson Orin Nano. New docker containers are built on the dusty_nv/jetson_containers repository (which is cloned into the dev/repos/ directory). As a result they build ROS 2 Humble from source, which means RoboStack is no longer used on-car.
The ZED ROS 2 driver now has full compatibility.
New ZED SDK is utilised, which provides AI assisted depth images as well as tracking, pointcloud mapping, and GPS integration.
Currently using the PyTorch implementation of YOLOv5. If this is too slow we can optimise for TensorRT.
[required] Documentation
Driverless/Software/Infrastructure/QEV-3 Onboard Systems
[optional] Usability concerns or breaking changes?
Very much a breaking change for any old camera driver settings.

Note: would be worth giving the ROSCube another rebuild too
Potentially we move away from Robostack on-car and just keep in our dev environments, as we are already using docker for environment recovery.


container: Fix container semantics

This pull request ensure the container pointers held by the pod structure are not duplicated. This is needed to avoid a container pointer modification not to be taken into account because the wrong pointer is used.
Fixes #588

LGTM


bug: ABI not creating types when importing as JSON

Is there an existing issue for this?

[X] I have searched the existing issues

Package Version
0.7.5
Current Behavior
Functions that rely on the ABI are currently not detecting the correct types when the ABI is imported as JSON. This happens specifically in useContractRead and usePrepareContractWrite but I haven't tested others.
Expected Behavior
I would expect the JSON import to be correctly picked by the hooks so that function names and arguments are typed.
Steps To Reproduce
ERC721.abi.json with other functions omitted for brevity. See CodeSandbox for full JSON.
[
  {
    "inputs": [
      {
        "internalType": "uint256",
        "name": "id",
        "type": "uint256"
      }
    ],
    "name": "ownerOf",
    "outputs": [
      {
        "internalType": "address",
        "name": "owner",
        "type": "address"
      }
    ],
    "stateMutability": "view",
    "type": "function"
  }
]

Then in my component:
import { useContractRead } from "wagmi";
import { BigNumber } from "ethers";
import { useIsMounted } from "../hooks/useIsMounted";
import ABI from "../ERC721.abi.json";

export default function IndexPage() {
  const isMounted = useIsMounted();

  // Return type of `data` is not defined correctly
  const { data: ownerOf } = useContractRead({
    // Deadfellaz as an example
    address: "0x2acAb3DEa77832C09420663b0E1cB386031bA17B",
    abi: ABI,
    // Should "fail" since `ownedOf` is not in the ABI, but `ownerOf` is.
    // Is typed as `string | undefined`.
    functionName: "ownedOf",
    // Also none of these args are typed
    args: [BigNumber.from(1)],
    watch: true
  });

  return (

Owner of token ID 1: {(isMounted ? ownerOf : null) ?? "-"}

  );
}

Link to Minimal Reproducible Example (CodeSandbox, StackBlitz, etc.)
https://codesandbox.io/s/cold-sound-f24lbx?file=/pages/index.tsx
Anything else?
I'm not sure if I'm doing something wrong when importing JSON, but I can't add as const since it's imported and not a literal. I've checked and copy pasting the ABI as a literal and adding as const works as expected.
My use case is a monorepo with foundry. I'd like to be able to import the ABI directly from the foundry output folder using a pnpm monorepo e.g.
import MyContractABI from<EMAIL_ADDRESS>
Here's my tsconfig.json if it helps:
{
  "compilerOptions": {
    "target": "es5",
    "lib": ["dom", "dom.iterable", "esnext"],
    "allowJs": true,
    "skipLibCheck": true,
    "strict": true,
    "forceConsistentCasingInFileNames": true,
    "noEmit": true,
    "esModuleInterop": true,
    "module": "esnext",
    "moduleResolution": "node",
    "resolveJsonModule": true,
    "isolatedModules": true,
    "jsx": "preserve",
    "incremental": true
  },
  "include": ["next-env.d.ts", "**/*.ts", "**/*.tsx"],
  "exclude": ["node_modules"]
}

Appreciate you putting together a solid issue! Unfortunately TypeScript doesn't support importing JSON as const right now so this doesn't work. Without giving too much away, watch out for a tighter integration between wagmi and Foundry/Hardhat in the future.

Yeah I landed on the same issue in TypeScript 😞. Appreciate the response though, and looking forward to the tighter integration.
For anyone else that ended up here, I decided to write a temporary shell script to take the contents of a Foundry ABI JSON file and save them to a .ts file exported as a const. You can see it in my template repo here.

I also created my own function to prepare Type Script files with abi as constant using Hardhat. In deploy script is function saveAbi to do it.


Vishal/more openapi changes

Removing height as require for the Get Block by height call since height can be absent if start_height and end_height query params are specified. 2374431dace71bd22901b6d7becd9a14375ef26e
The select query param lets you unselect any field in the response hence removing all the require for elements in the response object. 9a99bbf
For the GetCollection by ID call, it will be expensive to look up all the transactions for the collection and the response could be huge. So made a change to make the transaction in the response as expandable. If the query param specifies ?expand=transaction only then will the transactions be looked up and populated in the response, otherwise the _expandable field will list all the links (e.g. /v1/transactions/1234556999999) in an array thus skipping the extra lookups for transaction. 93d7a6d

Once y'all approve the PR, I will also include the generated files :)
Description

For contributor use:

[ ] Targeted PR against master branch
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Updated relevant documentation
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Added appropriate labels

@vishalchangrani are we making the change to include transaction expandable on collection as part of this PR or another?

Another comment I have is that we need to keep in mind removing required from response fields makes them have omitempty json tag, so we just need to be mindful if there are any values that should be present even if empty.
I personally don't think there is a problem with that (maybe the only one could be empty args on tx but not even that). However this will work nicely with collection being extended with transactions field, since if now we add transaction response with only IDs on collection it will show empty fields too which is not correct.

@vishalchangrani I'm not so sure about the change to collections. See the example I originally made on how this should be done: https://www.notion.so/dapperlabs/REST-API-Design-9594c690c8a342e383e1d372b5860116#ff2ec5c8469748ac859b60d195ae23ac

@vishalchangrani are we making the change to include transaction expandable on collection as part of this PR or another?

It is part of this PR - the third commit.

@vishalchangrani I'm not so sure about the change to collections. See the example I originally made on how this should be done: https://www.notion.so/dapperlabs/REST-API-Design-9594c690c8a342e383e1d372b5860116#ff2ec5c8469748ac859b60d195ae23ac

So in that example (copied below), the transactions are looked up by default (if no expandable is specified). That is something I am trying to avoid since it would have a performance hit since each of these transactions are individually looked by by their ID. Hence the proposed change, where by default only the trasnaction link is returned and only if expand=transaction is set then the actual transaction lookup takes place

// GET /collections/{id}
{
"id": "5491f56466ad8482e8f4393be0a00052f53987346b5ae8a355bf905d35190c3e",
"transactions": [
{
"id": "d83a3bfe450ac7c91ce489d47678d0ea55ab29dda265a603f0b9dd6e6635234f",
"script": "Jlfs7ZIXBPTbAElVzMsXkoUcI6EM1pP1+PegMYHFBflKfpHqpPLS2bFApQ7wpB/bOKuRB2+I0gr4IDwMmCZjJzEj9MBWRV/ZczM6/Qk82aIJSwGHR3cYQ76ArohmK5qs+4pof84Vd+aZ4LzkQOUOzf8zvpQlt7iKt6R22qxnllgYungG3qIzNXFXhfk/pCQFt/SLcAtOIu5Eu9pbjF0Sk7RYJdHGpRLvye/pDn1rq5C6pO7PydfIgMdqJDqrto7d+6o6T/oxozJiqL27EDFhKR3i061RYZ5Sza0YW6Phur5oRXNiHJlRAkS7zg3VCnb0d3rxvqh1YezwJS6hEg6hkvsovVWfS78s/99/POTgxbIZrz/sTTLGIRy9xjKf22f/dBPJOnfkD4pnzkP/LiuuhxcOdowbE/d1k8WXCFgXUBmSpbeWRr7h5qxutMwMoZ/XsPJc2ES6wNbj2B4BpF/um7EkVDqQv+dsNZjHqqbSUIh5rq5cOR8Dz11EVdsTkisjd9umC4jUkf8BdeR5cT/j4k1JBAzkmEWf1YZS/NUF/FD8ESOB15pB6OEYfBbs8GM9ZfpyRbvdYMKDnUjmttPf2VTL9qa2Kn9QybyuL2qmPMdODF97hmlEiwIwZIlsq1clUb269kD1zITRptLZkkS37qc5NDQ=",
"arguments": ["9qxmdg==", "Ej7v3+KQGenEvjUpVOe8Pg==", "0D6uxQ+ZFdg="],
"reference_block_id": "2a254dd82f13557b4ea069c6ad141221374ebfda90d739fd84c9e16c86410120",
"gas_limit": 100,
"payer": "3769b49731633aec",
"_links": {
"self": "/transactions/5491f56466ad8482e8f4393be0a00052f53987346b5ae8a355bf905d35190c3e"
},
"_expandable": {
"proposal_key": "",
"authorizers": "",
"payload_signatures": "",
"envelope_signatures": "",
"result": "/transactions/5491f56466ad8482e8f4393be0a00052f53987346b5ae8a355bf905d35190c3e/result"
}
}
// ...
],
"_expandable": {},
"_links": {
"self": "/collections/5491f56466ad8482e8f4393be0a00052f53987346b5ae8a355bf905d35190c3e"
}
}

@vishalchangrani I do understand what this tries to solve, I just don't think this is quite the right way to solve it. I would prefer one of two approaches here:

We keep the spec the same, but we add additional documentation to GetCollectionByID API description to call out that:

It is recommended to always use select=transaction.id unless the transaction details are actually needed. Looking up the full transactions for a collection is expensive and will result in a slower response.

This is the preferred approach.
2. We can make the default expandable fields for a transaction different when it is returned as part of a Collection. In particular, when part of a Collection we can have only the ID field expanded by default, and all other fields should be in the _expandable object

@vishalchangrani I do understand what this tries to solve, I just don't think this is quite the right way to solve it. I would prefer one of two approaches here:

We keep the spec the same, but we add additional documentation to GetCollectionByID API description to call out that:

It is recommended to always use select=transaction.id unless the transaction details are actually needed. Looking up the full transactions for a collection is expensive and will result in a slower response.

This is the preferred approach. 2. We can make the default expandable fields for a transaction different when it is returned as part of a Collection. In particular, when part of a Collection we can have only the ID field expanded by default, and all other fields should be in the _expandable object

I actually think that people will generally not take heed of the note and will call GetCollectionByID without query params. The default option should be the conservative one, that avoid expensive lookup and then if the client really know what they are doing, they can pass in the _expandable.
Choosing fields of an object by _expandable makes it work more like the select field and behave differently to how other expandable fields work where the expandable refers to complete objects instead of individual fields of an object.

I view the transactions within a collection analogous to payload or execution result within a block response, both of which as marked as expandable. If not specified, we only return the links for payload and execution result. This is the extension of the same concept with the only difference being it is an array of expandable links instead of just one as in the case of a block response.


jvm-target 17 isn't supported in compiler opts

I get an error that says:
error: unknown JVM target version: 17
Supported versions: 1.6, 1.8, 9, 10, 11, 12, 13, 14, 15, 16

It passes these arguments just on to the kotlin executable using the -X flag. So you may want to check what version of kotlin is running in your shell, and if it supports 17 as target.

I didn't realize that. Updating my local Kotlin to 1.6.0 fixes the issue! Thank you!


Implement PIP_TRUSTED_HOSTS logic from importing requirements to specified --index URL

Is your feature request related to a problem? Please describe.
The larger problem I'd ideally like to solve is to have newly created Pipfiles contain the correct source information and not be required to first fail, to have to edit the Pipfile, and then to attempt again.
Specifically, when adding an index via command line, if the index URL begins with https, verify_ssl is automatically set to true. However, if using private indices, this may be neither necessary nor preferred.
Describe the solution you'd like
The logic currently used when parsing requirements files in core.py that checks environment variable PIP_TRUSTED_HOSTS when determining the value for verify_ssl could also be implemented with index URLs specified at the command line.
Describe alternatives you've considered
Alternatively, one could pre-generate a blank Pipfile template with the correct sources and not use pipenv to create Pipfile. This would not be preferred.
Additional context
I'm happy to make the simple changes and create a pull request (see kalebmckale/pipenv@7c5dcde). I've already successfully tested this locally on my system, but I understand it needs to run though the full gamut of possible platforms, etc.

@kalebmckale Would you still be interested in taking on any fixes related to this?

@matteius I would love to be able to confidently create pull requests, but unfortunately I don't have a good development environment at home. Right now, I really just do programming on my iPad, and iSH isn't quite there with its functionality although it's pretty nice to have. I think the latest issue I ran into was the inability to use pyenv to install other versions of Python because iSH has yet to figure out (or perhaps just haven't released a fix to) shell redirection, i.e. <(...). tl;dr: I'm happy to do the coding, but I cannot really test it at home.

@kalebmckale opening a PR that is editable by the maintainers would be a good way to capture your ideas and possibly allow us to push them across the line.   There is so much going on for me that pipenv is a slice of it, and while I'd love to get the issue reports under 100 in the next year, it will require as much help as possible.

@matteius Okay, I think I did it correctly. It's technically my first PR on GitHub. I wasn't sure what extension to use for the news item but decided to go with feature.

@matteius @oz123 This can be closed / resolved?

@matteius @oz123 I figured out how to close the issue. So embarrassing! 😅


Project name with national characters

Hi all,
I use OpenRefine 2.8 on Linux with Oracle Java Version 8 Update 151.
If I create a new project with a name that contains special characters, then I restart OpenRefine, so I see only "???????".

Exact. If I create a project from the "Łukasz.csv" file, it is saved as "ukasz csv". If I then rename it "Łukasz", the change is replaced at the next start with "?ukasz".

Seems it only happens when rename it from project page. It works fine when do it from the about link. that is weird.

No. I tried it. The same problem.

I can't replicate this error with OpenRefine 2.8 (Linux version, running on macOS High Sierra) and Java:
OpenRefine >>java -version
java version "1.8.0_72"
Java(TM) SE Runtime Environment (build 1.8.0_72-b15)
Java HotSpot(TM) 64-Bit Server VM (build 25.72-b15, mixed mode)
I do find that non-ascii characters are dropped from file names on import and export (presumably not trusting the OS to necessarily handle filenames containing non-ascii nicely), but if I name my projects with non-ascii chars, then that seems to work fine within OpenRefine

If the problem is noticed after a restart, it could be that the issue occurs when the metadata is written to disk?

I use OpenRefine in Docker container. Dockerfile attached.
Dockerfile.txt
I think the container use UTF8.

I can reproduce under windows 10.
@rezabj can you please run a "locale" command in the environment running the OpenRefine

I can reproduce using that Dockerfile.
If I set the lang/locale to C.UTF-8 the issue goes away
Find locale on starting the Docker container:
root@37c04a8bf856:/opt# locale
LANG=
LANGUAGE=
LC_CTYPE="POSIX"
LC_NUMERIC="POSIX"
LC_TIME="POSIX"
LC_COLLATE="POSIX"
LC_MONETARY="POSIX"
LC_MESSAGES="POSIX"
LC_PAPER="POSIX"
LC_NAME="POSIX"
LC_ADDRESS="POSIX"
LC_TELEPHONE="POSIX"
LC_MEASUREMENT="POSIX"
LC_IDENTIFICATION="POSIX"
LC_ALL=

Set lang to C.UTF-8 and re-check:
root@37c04a8bf856:/opt# LANG="C.UTF-8"
root@37c04a8bf856:/opt# export LANG
root@37c04a8bf856:/opt# locale
LANG=C.UTF-8
LANGUAGE=
LC_CTYPE="C.UTF-8"
LC_NUMERIC="C.UTF-8"
LC_TIME="C.UTF-8"
LC_COLLATE="C.UTF-8"
LC_MONETARY="C.UTF-8"
LC_MESSAGES="C.UTF-8"
LC_PAPER="C.UTF-8"
LC_NAME="C.UTF-8"
LC_ADDRESS="C.UTF-8"
LC_TELEPHONE="C.UTF-8"
LC_MEASUREMENT="C.UTF-8"
LC_IDENTIFICATION="C.UTF-8"
LC_ALL=

Once I've done this, when I use non-ascii chars in the Project name it persists through the metadata save process

That's what I thought. But ideally it should be handled from the application level

It's help me. Thx.

Exported file loses Japanese characters in its filename, only extension. This also happens in case of non-ascii characters like à, è, ù.
For now, we should avoid non-Ascii Project names.

@isaomatsunami Are you also using OpenRefine 2.8 as the original poster? which version?

@isaomatsunami Please create a new issue describing the problem that you are having with full details of versions of OpenRefine, operating system, Java, etc
@rezabj This is likely is another, earlier, variant of same bug that caused #2543 #2544 #2627. It is fixed by #2657 which specifies UTF-8 for the workspace.json file on all platforms and doesn't depend on the locale being set to UTF-8. The fix will be in 3.4.

Sorry for my inadequate explanation and I should have reported earlier.
I tried with both OR3.4beta on current Mac OS 10.15 AND java-embedded OR3.4beta on WIndows10.

When starting project with file with Japanese name, all Japanese characters are lost in suggested (initial )project name, usually leaving only extention such as csv

So, I usually input manually a new project name in Japanese, like "選挙2020", election. The project is safely saved. the project list shows the Japanese project name perfectly. Reloading the project works fine.

When exporting csv/xls/html/odf, Japanse characters in exported file name are lost. BUT exporting project ( 選挙2020.openrefine.tar ) works fine, with Japanese characters intact.

Presumably this has been happening for a long time
I usually use ascii project name (is it bad habit?). But when I was teaching OR, they wanted to use original name.

On project creation we were dropping all characters not in the set [A-Za-z0-9] (ie replacing them with spaces), collapsing multiples spaces and trimming leading and trailing whitespace. I've changed this to just replace the characters .-_ with spaces, but would be willing to listen to arguments to either add to that list or get rid of the "cleaning" altogether.
On export multinational characters and punctuation was dropped and spaces were replaced with dashes. I took out the first part, but left in the space => hyphen conversion. My thinking is that modern file system are less restrictive about filenames and users can adjust their project names to be legal filenames on their operating system if necessary.
There are similar issues in the SQL exporter and the schema alignment, but I'm not sure of the requirements for those, so I'm not going to mess with them right now.

👍 drop the "cleaning" for the Project name.  Folks should be able to use what is available for their OS, with Unicode support now for the 3 major ones.

@tfmorris Thank you for quick response. I understand now its historical reason.
I feel sorry for my lack of ability to dig into code directly. Thank you again.

Sorry, I just discovered that this got yanked from the 3.4 release at the last minute, so it won't be available until 3.5.


Create Linux implementation

Linux version is TODOs and NotImplementedExceptions

If it's possible to do, we should do it in File.Copy itself. https://github.com/dotnet/runtime/pull/79243#issuecomment-1560324824

I was very happy to find this library and after reading the Readme I was sure it will work but then I run into this exception on Linux.
Exception has occurred: CLR/System.NotImplementedException
Exception thrown: 'System.NotImplementedException' in CopyOnWrite.dll: 'The method or operation is not implemented.'

at Microsoft.CopyOnWrite.Linux.LinuxCopyOnWriteFilesystem.CopyOnWriteLinkSupportedInDirectoryTree(String rootDirectory, Boolean pathIsFullyResolved)
If the library does not have Linux support yet, it should be documented up-front.
@danmoseley adding the COW feature to File.Copy was even better as this was the first place I was checking for COW support.

Added a note in README about the missing support.

On Linux, you should just pass through to the BCL implementation of File.Copy(). It is copy-on-write enlightened since .NET 7: https://github.com/dotnet/runtime/pull/64264.

That are awesome news and a reason to change .NET 6 to 7 on my Synology box. Thanks for the information!

Closing issue as unneeded. README.md updated with info about possible Windows future plus Mac and Linux.


bug(menu): When opening menu with keyboard, focus is set to the first item, but focus origin is set to "program".

Motivation
When opening a menu with keyboard as demonstrated in the repro steps below, focus moves to the first menu item (which is correct), but the focus origin is set to "program". This is a problem because some Angular Material consumers (in particular products internal to Google) have styles that depend upon the appropriate focus origin class. In my case, I have strong focus indicator styles that are only supposed to appear when the class .cdk-keyboard-focused is present. When opening a menu with a keyboard, I expect my strong focus indicator to render on the first menu item, but it does not.
Note that this is a problem in both the old Material and new MDC menus.
Problem & Solution
The problematic line is here. Menu only tracks whether it was opened by a "mouse" or "touch" focus origin, and if it is neither of those, it defaults to "program". The easiest solution seems to be to update menu to also track "keyboard" focus origin. The _handleKeydown event handler will need to update the _openedBy focus origin to "keyboard". I'm open to other ideas as well.
Reproduction
Steps to reproduce:

Go to https://stackblitz.com/angular/xkpoabmbqpb?file=src/app/material-module.ts
Open the menu with the "Enter" key.
Observe that the class .cdk-program-focused is present on the first menu item.

Expected Behavior
I expect the class .cdk-keyboard-focused to be present on the first menu item.
Actual Behavior
Observe that the class .cdk-program-focused is present on the first menu item.
Environment

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

In most places we treat program focus in the same way as keyboard in the CSS selectors. We should do the same here.

Sorry, it's not clear to me what fix you are proposing @crisbeto.

I was under the impression that this issue was about the focus indication not showing when the menu is opened using the keyboard, but that doesn't seem to be the case (https://github.com/angular/components/blob/master/src/material/menu/_menu-theme.scss#L37). Are you trying to use the focus classes for your custom styling?

Yeah. Internally in our product, we have some styles that only show the strong focus indicator when .cdk-keyboard-focused is present on the body (we have a single, global FocusMonitor with FocusMonitorDetectionMode .EVENTUAL that is attached to the body that we use to determine input modality).
IIRC, there was a reason we don't show the strong focus indicator on program as well. In our app, we don't really care about program because, given we have a single FocusMonitor with FocusMonitorDetectionMode .EVENTUAL, essentially all focus events are attributed to either mouse, touch, or keyboard. I could investigate showing the strong focus indicator on program, but I'm guessing I'll encounter some bugs and it seemed easier to contribute a fix for menu to AM.


Ran swiftformat . on main. No other changes.

Motivation:
Have main pass the soundness checks.
Modifications:
CONTRIBUTING.md recommends running the ./Utilities/soundness.sh script before making changes to the project. This script runs the swiftformat . command which currently modifies 34 of the files on main with no other changes to the project.
Result:
Bring main up to formatting requirements specified by CONTRIBUTING.md.

@swift-ci test


Enhancement: [select]="true"

When invoking inline editor via the API, one can select the editor content by passing .edit({ select: true }). It would be great to be able to be able to specify this behavior when the editor is invoked from a click when [editOnClick]="true" either with an attribute binding like [select]="true" or 							 [config]="{ select: true }".

Thanks four your feedback :+1:


Ghost Lines - Graphic character sheet

A new sheet for Ghost Lines that better matches the print character sheet, with sheetworker to convert old attributes to the new sheet.

@wing-it This PR added a whole bunch of Windows link files. I am quite sure this does not work as intended.

@kreuvf You're correct, thank you for reminding me, I meant to come back to this. There were a bunch of those link files, which I have removed, but all of the links in the sheet itself are to imgur, and everything seems to work as intended, at least as far as I can tell.
That being said, @timdenee we do recommend that the images themselves be added to the repository and that the sheet link to them on github. Aside from being convenient, this makes it easier for others if they want to contribute.
Your sheet seems to work fine as is, but if you would find the time to submit another pull request with the image files and updated links, we'd appreciate it!


Release v0.4.0

Two things left to do:

[] Bump the version in the makefile
[] Add a makefile rule that depends on cross and uploads to GCS

Seems like the binaries are not available in the GCS bucket.

The github release paths are incorrect
They are container-diff-amd64-linux, when they should really be container-diff-linux-amd64
cc @aaron-prindle @nkubala

updated, we recently renamed the binaries but did not update the release page.  This should be fixed now, thanks for catching this

https://storage.googleapis.com/container-diff/v0.4.1/container-diff-darwin-amd64
https://storage.googleapis.com/container-diff/v0.4.1/container-diff-linux-amd64
https://storage.googleapis.com/container-diff/v0.4.1/container-diff-windows-amd64.exe


deps: update to<EMAIL_ADDRESS>

I am able to reproduce this. Thanks for reporting.

The error originates in a place where we try to validate the ID of the implicit root:

I will need to release another version of dmn-js to fix that.

The error has been already fixed but we haven't updated dmn-js-properties-panel 🙈

As there are some pending changes in dmn-js-properties-panel, I will release a new version and integrate it here.

@MaxTru @Skaiir It's ready for review.

Works now - approved


Does it support Arabic

I have 50k high quality Arabic dataset,is possible to train the model on Arabic?

See comment here #6

I've added some initial pointers to this here: https://github.com/metavoiceio/metavoice-src/issues/70#issuecomment-1957337895

Hey @Qt4arab , we've just published an initial approach for finetuning the last N transformer blocks of the first stage LLM. Best to play around with the hyperparams in finetune_params.py as we didn't determine the optimal set.
Next step to improve finetuning effectiveness is to have LoRA adapters for the first stage LLM which is being worked on here.


Readme missing for v<IP_ADDRESS>

https://www.npmjs.com/package/ngx-slimscroll/v/9.0.0 is reporting that the readme cannot be found.

The license is missing in NPM.


Deprecation warnings

Thanks for porting it to Craft 3! There are several deprecation warnings showing in the CP for this plugin.

hey @darylknight. Thanks for logging this; has been on our backlog for awhile. Let us tackle this weekend before the holiday hits and we'll issue a new release.

craft.config.get() has been deprecated. Use craft.app.config.general.setting instead.

Thanks Rob - yeah, I'm not seeing these anymore either after updating.


nginx-slim: bake daemon off option into image

To use ningx with docker, you need to set daemon off.
So, people who run this image end up adding these 5 lines to pod spec:
        command:
        - nginx
        args:
        - -g
        - "daemon off;"

Would it make sense to add:
RUN echo "daemon off;" >> /etc/nginx/nginx.conf
to the Dockerfile so that people don't have to add those lines?

@aledbf do you maintain contrib/images/nginx-slim/?

@erictune yes. The CMD already contains daemon off but I can that echo in the Dockerfile

Oh.  Maybe that is good enough.
I was looking at this example:
http://kubernetes.io/docs/user-guide/petset/
which has this yaml https://raw.githubusercontent.com/kubernetes/kubernetes.github.io/release-1.3/docs/user-guide/petset.yaml
and was thinking it could be made shorter.

@erictune I just created https://github.com/kubernetes/kubernetes.github.io/pull/827

Thanks, @aledbf


feat: Enable TotalBytes metrics

Signed-off-by: Saravanan Balasubramanian<EMAIL_ADDRESS>

 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.

Can you please submit a PR to show this in the UI please?