Why will upickle convert Option[T] to array [...]?

I found if there is an option value in case class, it will be converted to [] in js
case class User(name:String, age: Option[Int])

val user1 = User("Free", Some(100))
val user2 = User("Wind", None)

Will be converted to:
{ "name": "Free", "age": [100] }
{ "name": "Wind", "age": [] }

Why use array here rather than:
{ "name": "Free", "age": 100 }
{ "name": "Wind" }

The later is the one I usually do with other scala json libraries

This sounds reasonable, but how would you encode Some(None): Option[Option[Int]]?

Never thought of that. You are right, it's reasonable to use array for Option.
But for the json readers, the array type may confuse them because they may think that value can have multiple items. Is there any way to customize the format?

No, unfortunately, it is not possible to customise the format. I agree that uPickle should provide an option to change the default behaviour. You could have a look at https://github.com/tindzk/upickle/commits/change-sumencoding or https://github.com/lihaoyi/upickle/pull/65 on how to introduce a configuration object. However, I'd prefer a solution with the Cake pattern, see https://github.com/lihaoyi/upickle/issues/59#issuecomment-72096982.

Closing this for now, feel free to talk on the other issues

@tindzk,

but how would you encode Some(None): Option[Option[Int]]?

There is a simple answer on that. Some(None): Option[Option[Int]] should be encoded as [[]] when it is a standalone value or a part of a collection, but if it is a part of case class:
case class Weird(value: Option[Option[Int]])

then the "outer" option should be encoded as a presence or absence of the value, while the inner option should be encoded as an array:
write(Weird(None))       shouldEqual """{}"""
write(Weird(Some(None))) shouldEqual """{"value":[]}"""
write(Weird(Some(1234))) shouldEqual """{"value":[1234]}"""

This does require that the macro-based serializer knowns how to handle optional fields, but it is not impossible and certainly makes sense.

This seems like a good approach to me. We should reopen the ticket.

Scala newb here, so this is probably wrong, but wouldn't:
write(Weird(Some(1234))) shouldEqual """{"value":[1234]}"""
in the case above be:
write(Weird(Some(Some(1234)))) shouldEqual """{"value":[1234]}"""
I'm also running into this issue, so it would be great to get this kind of Option handling in there.

@tindzk sure I can re-open it. I probably won't put in any work into fixing this, but now that uPickle's configurable via instantiating your own custom bundles it'd be cool if someone created a bundle/trait with this behavior

Closing out of inactivity

This works writer is applied for Some(v) :
  implicit val optHavingWriter = upickle.default.Writer[Option[HavingSpec]]{
    case Some(v) => writeJs[HavingSpec](v)
    case None => Js.Null
  }

This one too :
  implicit val optBooleanWriter = upickle.default.Writer[Option[Boolean]]{
    case Some(v) => writeJs[Boolean](v)
    case None => Js.Null
  }

But these are not :
  implicit val optStringWriter = upickle.default.Writer[Option[String]]{
    case Some(v) => writeJs[String](v)
    case None => Js.Null
  }
  implicit val optIntWriter = upickle.default.Writer[Option[Int]]{
    case Some(v) => writeJs[Int](v)
    case None => Js.Null
  }

Unfortunately generic types doesn't work at all, so I have to list a custom writer for each type :
  implicit def optWriter[T : Writer] = upickle.default.Writer[Option[T]]{
    case Some(v) => writeJs[T](v)
    case None => Js.Null
  }

It would be cool if it worked even for String and Int ... any idea ?

You actually can write a generic replacement for the Option Reader and Writer.  This trick is that you need to shadow the one that comes with upickle.  So, you can do it with the following:
  implicit def OptionReader[T: Reader]: Reader[Option[T]] = reader[Js.Value].map[Option[T]]{
    case Js.Null => None
    case jsValue => Some(read[T](jsValue))
  }
  implicit def OptionWriter[T: Writer]: Writer[Option[T]] = writer[Js.Value].comap {
    case Some(value) => write(value)
    case None => Js.Null
  }

This only works because The Reader has the name OptionReader and the Writer has the name OptionWriter.  These are the names of instances provided by upickle.  Since we are providing new ones with the same name, the ones provided by upickle will be shadowed and ignored.

One common use case would be to read {"a": "x", "b": "y"} to case class MyCC(a: String, b: String, c: Option[String]) so that it matches MyCC("x", "y", None).

Did anyone ever come up with a good way of doing this such that writing a Some(obj) is not written as an array AND None is not written at all? @asdcdow 's solution, and the solution in the docs makes Some write like the OP wants, but they also make None 's write as nulls.
I've tried playing with the OptionWriter source, but I'm just not understanding it's visitor pattern / array writing. It also seems like NoneWriter and SomeWriter do nothing?
I'm also serializeDefaults = true, which removes that route.
For others who can't figure this out, at least jq can hack around it:
jq 'walk( if type == "object" then with_entries(select(.value != null)) else . end)'

I guess I could write the json string, rewalk it and remove nulls, then write it again, but there must be a way to do this by overriding OptionWriter that I'm not seeing.

Should be fixed by https://github.com/com-lihaoyi/upickle/pull/598

Duplicate of https://github.com/com-lihaoyi/upickle/issues/528


Support lower camelcase

Many JavaScript users like to use camelcase in function name so I want to add the way to switch camelcase.
Changes

Added a instance variable for switching camelcase or snake_case

@yuku Hi. Sorry to bother you again, how about this?

@yuku Thank you for your review. Could you re-review this fixes?

Is it supports under Ruby 2.3.0? If not so, I add  gem.required_ruby_version = '>= 2.3.0' to js_rails_routes.gemspec because I want to use safe navigation operator. Although, it resolves by it will be usedObject#try method, However, support of Ruby 2.2 has ended.
@yuku WDYT?

Is it supports under Ruby 2.3.0?

It was. But it's ok to drop legacy Ruby versions.

Thanks to saying okay. I just sent divided pull-reqs.
How about these? https://github.com/yuku/js_rails_routes/pull/10

@sachin21 cloud you rebase this topic branch to latest HEAD?

@yuku Thanks for quickly merging! How about this?


'gem list --remote' does not respect proxy settings

Hi,
I am behind proxy. I have set proxy setting in sensu class
gem_install_options => [{ '--http-proxy' => 'http://<IP_ADDRESS>:2222' }]
all puppet runs takes very long time
Notice: Finished catalog run in 155.74 seconds
it looks like it does not pass install_options to the package correctly
as it stuck at  /usr/bin/gem list --remote ^sensu-plugin$ until it time outs
manual run:
time /usr/bin/gem list --remote ^sensu-plugin$ --debug -V
Exception `NameError' at /usr/lib/ruby/1.9.1/rubygems/command_manager.rb:177 - uninitialized constant Gem::Commands::ListCommand

*** REMOTE GEMS ***

Exception `Errno::ETIMEDOUT' at /usr/lib/ruby/1.9.1/net/http.rb:762 - Connection timed out - connect(2)
Exception `Gem::RemoteFetcher::FetchError' at /usr/lib/ruby/1.9.1/rubygems/remote_fetcher.rb:246 - Errno::ETIMEDOUT: Connection timed out - connect(2) (http://rubygems.org/latest_specs.4.8.gz)
sensu-plugin (1.2.0)

real    2m7.807s
user    0m0.504s
sys     0m0.056s

time /usr/bin/gem list --remote ^sensu-plugin$ --http-proxy http://<IP_ADDRESS>:2222

*** REMOTE GEMS ***

sensu-plugin (1.2.0)

real    0m1.547s
user    0m0.467s
sys     0m0.071s

Setting global proxy settings kinda "solves" the problem but I need it to be managed through module itself.
/etc/environment
 export http_proxy=http://<IP_ADDRESS>:2222

Karolis

Would you be able to take a stab at a PR for this?  I don't have a proxy server to test this with.

@karolisc the local sensu gem provider uses the gem provider as its parent:
Puppet::Type.type(:package).provide :sensu_gem, :parent => :gem do

We also explicitly take the install options parameters:
has_feature :versionable, :install_options

Now your issue seems to be with the actual gem list command. We use the upstream gem provider here:
https://github.com/puppetlabs/puppet/blob/master/lib/puppet/provider/package/gem.rb
And as far as I can tell, there's no ability to pass options to gem list:
 def self.gemlist(options)
    gem_list_command = [command(:gemcmd), "list"]

    if options[:local]
      gem_list_command << "--local"
    else
      gem_list_command << "--remote"
    end
    if options[:source]
      gem_list_command << "--source" << options[:source]
    end
    if name = options[:justme]
      gem_list_command << "^" + name + "$"
    end

    begin
      list = execute(gem_list_command).lines.
        map {|set| gemsplit(set) }.
        reject {|x| x.nil? }
    rescue Puppet::ExecutionFailure => detail
      raise Puppet::Error, "Could not list gems: #{detail}", detail.backtrace
    end

Adding this as part of the module seems out of scope for this particularly provider - I expect you'll have the same issues with the standard puppet provider as well? I'm going to close this, as I think it's a problem with the original puppet provider, not with the sensu gem one.
Please reopen if you think I'm in error.


test: tweak BNB chain test

Add comment and more explicit assertion.

Confirmed this is working on both sepolia and BNB mainnet. merging


Add Magic Leap to USERS.md

Adding Magic Leap as user of hubble as discussed with @xmulligan
Signed-off-by: RoCha<EMAIL_ADDRESS>

Thanks for the addition @romachalm !

this pr only modifies USERS.md so no need to run the full CI. marking as ready-to-merge.


Can no longer pass in transformTimeoutInterval as opt

Do you want to request a feature or report a bug?
Request a feature
What is the current behavior?
Unable to pass transformTimeoutInterval as opt
What is the expected behavior?
Would be great to pass this in via project.ext.react in build.gradle
Related issue reported under react-native project: https://github.com/facebook/react-native/issues/15401
Original commit that reverted option: https://github.com/facebook/react-native/commit/242ced150c4d44c68f97e2b0cf9722397d438004

We are currently rewriting the transformer, and will consider to fix this :) Closing for now as this specific issue isn't actionable by external users.


NMS.GEN.LOG.010 - Audit trail logging props module

NMS system shall have audit trail logs that includes but not limited to the following:

Any NMS configuration change related to business logic (refer #125 NMS.MA.CFG.001)

The audit trail log shall include event details, date/time of event and user credential information among others.

@kamknitt I split #142 I think what is in this ticket you are already working on

This just involves the domain objects in the props module

@kamknitt I'll take this one since I think you are currently working on the Location ticket


Poor translation to steamvr

I'm not sure if I am the only one having this problem, but the 'out' window is tracking everything really well, but in SteamVR the movement is inaccurate and jittery, is there any way I can fix this?

Detecting joint positions on an image is easy, but transfering that to a full 3d skeleton is much harder. There is a lot of information that needs to be guessed, mostly the distance of joints from the camera, but also the camera position, which way is down etc. This causes plenty of innacuracies, even when the 2d windows looks fine.
Still, plenty of things can be done to improve things, mostly ensuring a good camera position, good lighting, and well seen clothing. Copited from the readme:

Make sure that your entire body is seen on the camera, including your head!
Make sure there is enough light in the room. and there shouldnt be any light sources behind you, such as any bright windows. Try to get as much lighting as you can from the direction of the camera.
Mind what you wear: you need to be in contrast with the background, so dont wear clothes the same color as the background and avoid clothes with patterns.
Play with the smoothing and latency parameters a bit. To learn what exactly they do, we have some nice graphs here.

It can also help if your camera is placed about head high, pointing slightly downward.
In the end, plenty of these settings are dependant on your particular setup, so you may need to experiment a bit to get optimal performance.


pinocchio: 3.2.0 -> 3.3.0

Announce & changelog: https://github.com/stack-of-tasks/pinocchio/releases/tag/v3.3.0
Diff: https://github.com/stack-of-tasks/pinocchio/compare/v3.2.0...v3.3.0
Also removed postConfigure, since that issue was fixed in upstream at https://github.com/stack-of-tasks/pinocchio/pull/2416

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@ofborg build pinocchio pinocchio.passthru.tests


docs: clarify licensing

Apache V2 and MIT is almost same with respect to licensing the code.
However Apache V2 is more verbose and used by ecosystem projects
like Cincinnati. Hence moving to only Apache V2
Signed-off-by: Lalatendu Mohanty<EMAIL_ADDRESS>Closes: #143

@LalatenduMohanty thank for the cleanups. However I think #143 is just asking us to make the license more prominent, not to relicense the whole crate (MIT OR APL2 is fine, and better aligned with the larger Rust ecosystem).
I think you can follow the approach from other repos (this is a personal example I have) and mirror it here, like this:

Add a snippet at the bottom of README
Add a COPYRIGHT file
A a LICENSE-MIT and LICENSE-APACHE-2.0

Thanks @lucab . It is interesting that Rust ecosystem prefers dual licensing.

@LalatenduMohanty that is documented in the language guidelines. It shouldn't really matter for Cincinnati, as the consumer can pick which of the two it prefers.

@lucab Thanks, I have updated the PR as per https://rust-lang.github.io/api-guidelines/necessities.html#crate-and-its-dependencies-have-a-permissive-license-c-permissive


Fix fix tablet showing problem

tablets could not showed in explain string when selected partitions too large, so it need to be show in next line

run buildall

run buildall

run buildall

run buildall

run p0

run p0


fix a bug : the redirect code in HttpUrlSource is not working, becaus…

English is poor, hope you understand.
HttpUrlConnection use a flag -- "instanceFollowRedirects", to define whether the protocol will automatically follow redirects or not. And the default value is true. So all redirects are handled by  httpUrlConnection automatically. The redirect logic in HttpUrlSource have no chance to be run. We need to invoke setInstanceFollowRedirects(false) to get that chance.

@Labmem003 Thank you.

@JingweiWang my pleasure


Fix lottie web widget not to capture other view input events

https://github.com/EnsembleUI/ensemble/issues/1218
It fixes two issues

This allows to get mouse/input events for other views on top of it or on the next page in the region of the Lottie widget on the previous page

Also it fixes the onTap issue with HTML Renderer also

Please make sure to remove all the hacks from the Kitchen Sink code before merging this in.

I added only the captureWebPointer to all the custom widgets and removed that already


Plurality of error messages

Expected behaviour
When there is only one error message, it should be worded appropriately, something like:

This page has an error
This must be corrected to continue

Actual behaviour
When only one error is raised the wording is:

This page has 1 errors
These must be corrected to continue

Steps to reproduce the behaviour
On any screen that can generate an error, input invalid data and submit.
Screenshot

@ajmaddaford maybe this can be added as part of the work for the error messages card: https://trello.com/c/tBLC40PT/1529-error-messages-for-different-field-types


Add custom webview for importing via GPM

Task/Issue URL: https://app.asana.com/0/608920331025315/1207414915035673/f
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#5097  👈
#4602
#4601
#5096
develop

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @CDRussell and the rest of your teammates on  Graphite


Linux process doesn't terminate on exiting the launcher

Expected
Clicking on "Exit Launcher" results in launcher closing and terminal process terminating.
What happened
On clicking on "Exit Launcher" the launcher window closed, but the terminal process kept running.
Additional Info
OS: Linux Mint 18
Launcher Version: 4.1.1
How to reproduce

Start the launcher (run in terminal)
Configured it to not close on starting a game
Start a game
Exit the game
Click on "Exit Launcher"
Observe in the terminal, that the process doesn't stop.

I'm having trouble reproducing this when running from source.
But if it's related to the game runner thread — and it probably is, if it only happens after starting & exiting a game — it might well be fixed by #571.

I can't reproduce this as well. A freshly downloaded 4.1.1-Linux64 build exits as expected on my arch linux. Could this be a ubuntu/mint issue?

Exits fine for me as well - I think the underlying problem is the game process not closing properly, I've create MovingBlocks/Terasology#4240 to track that.


Improve GetPackedSize performance on arrays of blocks.

Currently the logic for getting the packed size is, not great.
For arrays of AeroBlock structs it will loop over each and add up the size, this is because AeroBlocks can have logic and variable sized arrays or more sub AeroBlocks.
Some of them don't and are of fixed size, we should have a step that checks if a block is a fixed size and if so record and use that in the GetPackedSize function to avoid having to loop over those when its not needed.
eg, currently the function for a block could output something like:
for(int idx0 = 0; idx0 < ArrayTest.Length; idx0++)
{
  size += sizeof(uint);
  size += sizeof(float);
}

and for an array of a know fixed size item like an int
size += sizeof(int) * 4;

The block example could be simplified to much the same as the above since the fields in it are of a fixed size as well.
This can be related to issues #2 and #3

Fixed in one of the past commits :>


Add Switzerland to country enum (EXPOSUREAPP-5282)

This PR adds the Switzerland flag to the list of countries for transnational exposure logging.
The flag should be shown in the country view in

Länderübergreifende Risikoermittlung Screen
Submission Flow "Ihr Einverständnis"
OnBoarding "Wie sie die Risikoermittlung ermöglichen"
Delta-Onboarding "Länderübergreifende Risikoermittlung" (added in Version 1.5)

The backend is not yet adding "CH" in the list of supported countries in the AppConfig. Therefore, you have to add the following:
else -> supportedCountriesList.toMutableList().apply { add("CH") } in Line 28 of CWAConfigMapper.
On most test environments, only DE is set as supportedCoutry. Therefore I used prod to test it.

The flag and name of Switzerland is there, but I got this dialog frequently on test environments.


Coexist with socket.io

Can it coexist with socket.io? Because after I define require('express-ws')(app,server), socket. io client receives disconnections due to "transport error" from server service that runs both express-ws and socket.io.

Answer is yes. Both libraries can coexist so long as both don't listen to the same tcp/udp port. The way to resolve this is to assign different ports for both websockets and use proxy service' directive feature to divert different websocket traffic to respective websocket.


Search within search inside file

When searching inside a large file I would like to be able to search between the results of my first search. So that I can firstSearch && searchInBetween in the search textfield.
I can press enter with firstSearch like it is know with existing functionality. If there are 4 instances of firstSearch - move to instance 3 with enter - and by appending with ... && newSearch to search between 3 - 4 instances of firstSearch. with a new phrase: newSearch
It would make the search more like a query. And the times you want to search for instances of && in the code it would be a problem. But something like this would help moving through a lot of code quickly.

I don't want to support special syntax in the search field, when people need to be able to just search using that syntax. It would be confusing


Publish Inexor Flex as debian packages (Rolling Release)

[x] Create a npm package (see #518)
[ ] Create a debian package using node-deb

https://www.npmjs.com/package/node-deb#ex-3

[ ] Dependencies (nodejs, npm, ...)
[ ] Post-Install-Script (for example download/update the necessary releases, media repositories and the user interfaces during the apt-installation)

[ ] Publish the debian package on https://github.com/inexorgame/inexor-flex/releases

This could be done better and for more platforms with snapd, publishing to the Ubuntu Store also..


Remove proxy url

Coverage remained the same at 82.989% when pulling 02a38895529fd28855b4a33bf29384f84d2e83d7 on oleksiyVeretiuk:remove_proxy into 2725551fba8262babd4ae0d19909a067411a3b1b on openprocurement:sandbox.


ImportWarning when importing zmq

Importing zmq within my scripts frequently gives the message:
ImportWarning: can't resolve package from __spec__ or __package__, falling back on __name__ and __path__
This doesn't happen in interactive mode, but e.g. the following script (filtering the warning above to cause an error to get the traceback)...
import unittest
class Test(unittest.TestCase):
    def test_1(self):
        import warnings
        warnings.filterwarnings('error')
        import zmq
unittest.main()

...results in...
Traceback (most recent call last):
  File "test.py", line 8, in test_1
    import zmq
  File "/Users/ahm/anaconda3/lib/python3.6/site-packages/zmq/__init__.py", line 34, in <module>
    from zmq import backend
  File "/Users/ahm/anaconda3/lib/python3.6/site-packages/zmq/backend/__init__.py", line 40, in <module>
    reraise(*exc_info)
  File "/Users/ahm/anaconda3/lib/python3.6/site-packages/zmq/utils/sixcerpt.py", line 34, in reraise
    raise value
  File "/Users/ahm/anaconda3/lib/python3.6/site-packages/zmq/backend/__init__.py", line 27, in <module>
    _ns = select_backend(first)
  File "/Users/ahm/anaconda3/lib/python3.6/site-packages/zmq/backend/select.py", line 33, in select_backend
    reraise(ImportError, ImportError("Importing %s failed with %s" % (name, e)), exc_info[2])
  File "/Users/ahm/anaconda3/lib/python3.6/site-packages/zmq/utils/sixcerpt.py", line 33, in reraise
    raise value.with_traceback(tb)
  File "/Users/ahm/anaconda3/lib/python3.6/site-packages/zmq/backend/select.py", line 26, in select_backend
    mod = __import__(name, fromlist=public_api)
  File "/Users/ahm/anaconda3/lib/python3.6/site-packages/zmq/backend/cython/__init__.py", line 6, in <module>
    from . import (constants, error, message, context,
  File "zmq/backend/cython/error.pyx", line 32, in init zmq.backend.cython.error (zmq/backend/cython/error.c:1228)
ImportError: Importing zmq.backend.cython failed with can't resolve package from __spec__ or __package__, falling back on __name__ and __path__

Reproduced in:
anaconda 4.3.1, python 3.6.0 on Mac OSX with zmq.zmq_version() => 4.1.6
anaconda 4.3.0, python 3.6.0 on CentOS7 with zmq.zmq_version() => 4.1.5

I suspect this is a Cython issue, where the generated modules only have the expected metadata for Python < 3.6.


newsreader rewrite has wrong test for content wrap.. (should be if not)

I found a bug in MagicMirror 2.15-develop
MagicMirror Version: Please let us now which version of MagicMirror you are running. It can be found in the package.log file.
2.15-develop
Description: Provide a detailed description about the issue and include specific details to help us understand the problem. Adding screenshots will help describing the problem.
old code says (!this.config.wrapDescription)? "noWrap":""
new code has if wrapDescription "noWrap" : ""

So this issue can be closed now @sdetweil ?


Continued work on widgets for controlling RE Manager

The PR reflects additional work on the widgets for controlling RE Manager:

widget for monitoring RE Manager status (QtReStatusMonitor).

Trying out your PR to see what it does, I found one spot where an empty catalog None crashes the GUI: https://github.com/bluesky/bluesky-widgets/commit/2e2dc1f68f55cc14987628aeb455205dd25cc8e5
Feel free to pick it up the commit/fix your PR if you like.

@klauer This PR is about 50% ready. It will take me at least a couple of days to finish it.

@danielballan Would you like me to include the corrections from https://github.com/bluesky/bluesky-widgets/commit/2e2dc1f68f55cc14987628aeb455205dd25cc8e5 in this PR or would you like to create a separate PR?

Thanks, @klauer. @dmgav I'll pick 2e2dc1f up myself in a separate PR since I'm working on that as well.

Once you have addressed or rejected the  comments, please self-merge at will.


Add RST packet processing fixes

This PR fixes several issues when processing RST packets, including original issue #154:

Server sends RST packet with token
Server sends NON '5.00' "Empty message must be empty" after the RST
Server sends ACK after the RST
In rare occasions client may send a RST message in response to RST
Request does not finish after receiving RST (cleanUp is not called, so the program just "hangs" without finishing)

Could anyone review the changes? @mcollina @neophob?
Also, I was unsure how should I handle a RST as a response - at first I did a separate req.on('reset') event, but then just slapped RST into response as it does not alter the API. Which do you guys think is better?

Hi @GiedriusM - looks nice. I will test this next week to see if there any side effects.

Thanks, I'll add you as a reviewer then.

Merged with the (hopefully) working fix. Tests seem to pass without any disturbances.

finally - could test with my app, LGTM!

Thanks, @neophob! I'll merge it once you validate the review.


Ablate on initialization

interested in:
A) changing layer_id + 1 to args.num_layers.
B) removing the line std = std / math.sqrt(2 * (layer_id + 1))

related: https://github.com/mlfoundations/open_lm/pull/225

We tested #225 at 1B and it seems to hurt downstream evals significantly, unfortunately.


renamed {task} to {gulp_task} in Gulp\Run. Fixes #337

Coverage remained the same at 43.167% when pulling 6b992721b445222c0fcf401fee4d0f2bf4d2b00d on bankiru:fix-fail-of-taskGulpRun into 55e6f896c857bddc58fcc9af48606d2b0b23b740 on Codegyre:master.

Thanks.


no-merge: try fixing CI

fail-fast: false in the github workflow strategy seems do be the trick to allow unstable to fail while stable completes: #46


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-yml-archive-non-versioning-archive/pull/153 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-yml-archive-non-versioning-archive.git

Docs Build status updates of commit 4ec6388:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

docs/TOC.yml
:white_check_mark:Succeeded
View

docs/index.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-3.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/sql-database-engine.png
:white_check_mark:Succeeded

docs/to-be-archive/level2/article3-2-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-3.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


it's works in dockerfile container image?

i'm creating a simple duckdb container with the commands sent by duckdb -cmd ".read startup.sql":
startup.sql content's
INSTALL httpserver FROM community;
LOAD httpserver;
SELECT httpserve_start('localhost', 9999, 'user:pass');
select 24;
the content of the dockerfile is:
FROM ubuntu:20.04
WORKDIR /app
RUN apt-get update && apt-get install -y wget unzip iptables nano curl
RUN wget https://github.com/duckdb/duckdb/releases/latest/download/duckdb_cli-linux-amd64.zip -O duckdb.zip
&& unzip duckdb.zip
ENV PATH="/app:$PATH"
COPY . .
CMD ["duckdb", "-cmd", ".read 'startup.sql'"]
EXPOSE 9999
when i execute the request inside the container, it's done, but, it's dont work outside!
is a problem of duckdb?
Obs: i check the firewall, but, the container is clean of others executors and the top has only duckdb process in "background"
Thanks!

Hello @qweralfredo
If you want the API run in foreground while in-process set DUCKDB_HTTPSERVER_FOREGROUND=1
For the same out-of-process you want to consider the reverse-twin of this extension quackpipe

thanks @lmangani!
But, its dont works, but, it's run perferctly in docker (local) and when i using ngrok tunnel access.
i'll recreate it!

is a problem of duckdb?

No, it works very well.
Dockerfile
# syntax=docker/dockerfile:1
# check=error=true

FROM docker.io/library/debian:bookworm-slim AS base

# Throw-away build stage to reduce size of final image
FROM base AS build

# Set environment
ENV DUCKDB_VERSION="1.1.3"

# Install packages needed to install DuckDB
RUN apt-get update && \
    apt-get install --no-install-recommends -y \
      ca-certificates \
      curl \
      unzip

RUN curl \
      --fail --silent --location \
      "https://github.com/duckdb/duckdb/releases/download/v${DUCKDB_VERSION}/duckdb_cli-linux-$(uname -m).zip" \
      -o ./duckdb.zip && \
    unzip ./duckdb.zip

# Final stage for app image
FROM base

# Copy built artifacts
COPY --from=build /duckdb /duckdb

ENTRYPOINT ["/duckdb"]

Then
bash
podman build -t my-duckdb-image .

podman run -ti \
  --init \
  -p 4000:4000 \
  -e 'DUCKDB_HTTPSERVER_DEBUG=1' \
  -e 'DUCKDB_HTTPSERVER_FOREGROUND=1' \
  my-duckdb-image \
  -c "INSTALL httpserver FROM community;" \
  -c "LOAD httpserver;" \
  -c "SELECT httpserve_start('<IP_ADDRESS>', 4000, '');"

(it downloads the extension, and won't output anything)
You can then use it with something like
curl -X POST -d "SELECT 'World' as Hello" "http://localhost:4000"

What a fantastic example thanks for sharing it with the community @vhiairrassary I'm sure other users will treasure it

No, it works very well.


Docs: How-to - Use Preset as a Superset backend

@bmtcril I don't think is possible right now. There are just too many specific things that we developed based on Tutor that I don't think this will be possible. Mainly because of the custom scripts

I agree, and I haven't been able to get a test account to try it out and get support so I'll close this one.


Address how to move from Uni to Multi and vice versa

I forgot if that's addressed already but allowing people to move between the types is probably necessary.

So Uni.createFrom().publisher(multi); would work

I think this one is already covered.

Uni from a Multi yes
Multi from an Uni not yet (coding in progress).

Fixed in #1


weave-225: [dns] caching (2/3)

#225: this pull request introduces a limited number of concurrent resolvers in the DNS server. Clients requests are distributed among these resolvers with no duplicates (ie, there will be only one resolution for a given name, even when multiple clients are performing the same query).
Performance tests with DNSblast show a significant increment in the number of requests that can be served per second as well as a reduction in the number of I/O timeouts to upstream servers.
Important: this PR depends on #429

Performance tests with DNSblast

I'm intrigued. Would you mind elaborating what/how you tested?

I tested performance of upstream queries, mostly for checking how the cache works. Starting WeaveDNS with
sudo weavedns -dnsport=1053 -fallback="<IP_ADDRESS>:53" -debug

and launching dnsblast with something like
dnsblast <IP_ADDRESS> 50000 100 1053

This would generate 100 reqs/sec of DNS A-queries for names with the form <random char><random char><random char><random char>.com. Invoking dnsblast again with the same arguments generates the same sequence of A-queries, a nice feature for testing the cache, even when queries following a Pareto distribution would be much more realistic...

My guess in the reduction of I/O timeouts to fallback servers is that the new mechanism reduces the number of concurrent queries to Google's servers, while the old mechanism generates too many requests and they probably drop packets from the same IP when they come too fast...

Closed until further notice...


drivers: wifi: winc1500: removed semaphore on accept.

This fixes #28953 where winc1500 driver blocks on listen.
Signed-off-by: Nicolai Glud<EMAIL_ADDRESS>

Your bug report talks about driver blocking on listen(), but how this PR is then fixing that as we are removing the lock in accept()?

The calls chain is:
Listen -> zsock_listen -> z_impl_zsock_listen -> zsock_listen_ctx -> zsock_listen_ctx (sockets.c:431) -> net_context_accept ->
net_offload_accept -> winc1500_accept.
This is done to install the accept callback as far as I know.

The calls chain is:
Listen -> zsock_listen -> z_impl_zsock_listen -> zsock_listen_ctx -> zsock_listen_ctx (sockets.c:431) -> net_context_accept ->
net_offload_accept -> winc1500_accept.
This is done to install the accept callback as far as I know.

Information like this would be good to have in the commit message because the change is not obvious how it fixes things. I just wonder how this has been working before.

To my knowledge, this has never worked. I could not get the sample running before I did the work in #28300.

Are you actively working on winc1500 chip in the future? I am asking because there is no maintainer marked for this driver in CODEOWNERS, so it would be rather easy to become a maintainer if you are interested about that.

I am working on a generic module for wifi and using the adafruit WINC1500 wifi shield for development.
What exactly does becoming a maintainer involve?

What exactly does becoming a maintainer involve?

The terminology seems to be have changed recently, the role is called now "Collaborator". See description here https://docs.zephyrproject.org/latest/contribute/project_roles.html#collaborator
Basically it would mean that you would need to review PRs for this driver, fix bugs if there are any and optionally develop new features if needed. So something like what you have already done.

Sure, you can add me for this device.

Sure, you can add me for this device.

Usually we do it so that the person would send a PR for this. So if you could add yourself to CODEOWNERS and MAINTAINERS.yaml file in relevant places and send a PR it would be great. So for example for MAINTAINERS.yaml file, you could create entries similar to es-WiFi is doing.

Information like this would be good to have in the commit message because the change is not obvious how it fixes things.

Could you enhance the commit msg, the patch itself looks ok.

@jukkar please review


Glorious Model I Wired Support

I've been searching for hours for a solution to have my mouse buttons identified. To my surprise still no support.
I play games disabled (missing left lower arm) and am in need to use additional mouse buttons to play games.
I just switched over to Linux and am having to dual-boot Windows for the time being, if this problem can be solved I would be overjoyed. Thank you for the current support!

Hi! This looks like a duplicate of #1327, if "Wired" is the same thing as usual "Glorious Model I".
What is the output of lsusb for you, @xanzoh?

Hey thanks for the response i was reading the linked issue you sent, I have the "Wired" version of the Model I, and i can confirm it's the same issue (same lsusb output too.) As of now I've unfortunately returned back to Windows.

Yeah, as that issue says, Model I uses different protocol from older Model {O,D} and new Model {O,D} Wireless mice.
libratbag doesn't support it, and neither do I know any other utilities that do.
Sorry for not being of any help.
Closing as duplicate of #1327.


Failure when dbConnection not created

Describe the bug
When calling sbt test command on local env without accessible atum_db, dbConnection init fails.
In the test method in the finally block, there is no check that dbConnection has been successfully initialized.

I don't recall what exactly is the problem here. If the DB is not available, it's fine to fail isn't it?

I don't recall what exactly is the problem here. If the DB is not available, it's fine to fail isn't it?

The problem is that the command test runs unit tests, which should be immune to the database's availability.

I don't recall what exactly is the problem here. If the DB is not available, it's fine to fail isn't it?

The problem is that the command test runs unit tests, which should be immune to the database's availability.

Wait, what? One writes DB tests, DB not available, test fails. Seems like a correct and expected behavior.

I did a re-test of the problem.
No DB created on local:

sbt test - PASSED
sbt testIT - FAILED as expected
sbt testDB - FAILED as expected
DB instance created on local:
sbt test - PASSED
sbt testIT - PASSED
sbt testDB - PASSED


Bug: News Feed Not Working for Export Markets

Issue Type
Bug
Priority
Blocker
Description
The News Feed for a few export markets is not functioning as expected. When accessing the feed URL for specific markets, no data is returned or an error occurs.
Affected Device Classes
Tablet, Mobile, Desktop
Affected Browsers/Devices and Versions
All browsers and devices
Steps to Reproduce

Open: https://main--macktrucks-com-do--volvogroup.aem.live/body-builder-news.json
Observe the issue (no data or error message displayed).
The feed is expected to return a JSON response with news data but does not.

Expected Results
The news feed should return a valid JSON response with relevant news data for export markets.
Actual Results
The URL does not return the expected JSON data. The News Feed appears broken or inaccessible for specific export markets.
Attachment

@Kannan-CM Is this all export markets or only a few? Please specify the markets. Thank you!

Another issue is that the news-list grabs an hardcoded link for the xml file and that needs to come as a property on the component.

the json file is empty, even if is already previewed/published, if is empty, the result JSON it's ok if it has 0 results

Waiting for meeting with Darin on 12/9/2024 FYI @BeckyMedlin @ZoeBeykirch @KJohnsn

Testing Completed. Everything works accordingly.


Add docker support

The docker-compose file to manage web and db containers
Uses supervisord in web container to manage bot and web processes
Web server uses uwsgi
Docker entrypoint script automatically runs migrations at start time if env DJANGO_MANAGEPY_MIGRATE is set to 'on'
Had to change BASE_DIR in settings.py to fix a bug with collectstatic at container build time

@fijter I noticed this PR is still open. Have you had a chance to review it? Are there any changes I should make to it?


Embedded TTF Fonts Not Working when indexToLocFormat is 0 (SHORT)

I am using the latest version of jsPDF: 1.4.1
I have also tried using jspdf.debug.js
I have been able to get the basic example to work with the sample font.
however, i have not been able to inject courier prime into the pdf
Steps to reproduce

Download Courier Prime from http://dimkanovikov.pro/courierprime/
convert to base64
open the working basic example code
replace sample font string with the bas64 encoded string
print a sequence of cyrillic letters
save PDF
open PDF

BUGGY RESULT: lots of exclamation marks

print a sequence of english text
save PDF
open PDF

BUGGY RESULT: some # signs

Similarly, I have had no success embedding
https://www.1001fonts.com/open-sans-font.html

I did a little more digging into the font files themselves.
For the working sample font, the indexToLocFormat is 1 [Meaning Long Offsets (32)] whereas for the two fonts which do not load, the indexToLocFormat is 0 [meaning Short Offset(16)]
That might be the issue.

I have confirmed with several fonts that TTF fonts which have indexToLocFormat 1 will display whereas those with indexToLocFormat0 do not display.
Here's a bit of perl which will dump out the TTF headers:
use strict;
use warnings;
use Font::TTF::Font;
use Font::TTF::Dumper;
die "Provide One Argument: Filename" if !$ARGV[0];
my $font = Font::TTF::Font->open($ARGV[0]);
print ttfdump($font->{'head'}, 'head');

Well Aras, who are certainly worth a beer and lots of thumbs up if you find a solution to this! :-)

@hdale
Well, there are alot of issues in the pipe... :-/

I am excited to see progress on this issue.
Thanks. just keep plugging away at it.

Any update on this? This is breaking some important functionality on my site as well :(.

fixed with #2019 which was adapted work of @briandevet <-- thank you very much.

Confirmed working! Thank you so much both @arasabbasi and @briandevet for the fixes.

I can also confirm that the fix works for us.
Thank you very much - greatly appreciated!


Not intercepting as expected

Hi,
I've been trying to get msw+playwright + playwright-msw to work but can't figure out what I'm either doing wrong or what isn't working.
My setup is this:

Vite.js build that is hosted with vite preview
MSW handlers generated by using msw-auto-mock and a swagger.json Open API documentation file

Example of one of these handlers:
export const handlers = [
      http.get(`/api/v2/fields`, () => {
         return HttpResonse.json(mocked body response);
     })
]

Playwright with custom fixture like you have in the README:

import { expect, test as base } from '@playwright/test';

import { handlers } from './mock';

import { http } from 'msw';
import type { MockServiceWorker } from 'playwright-msw';
import { createWorkerFixture } from 'playwright-msw';

const test = base.extend<{
  worker: MockServiceWorker;
  http: typeof http;
}>({
  worker: createWorkerFixture(handlers),
  http,
});

export { expect, test };

My playwright.config.ts
import { defineConfig, devices } from '@playwright/test';

import dotenv from 'dotenv';
import path from 'path';

dotenv.config({ path: path.resolve(__dirname, '.env.e2e') });

const PORT = 3001;

export default defineConfig({
  testDir: './src/e2e',
  timeout: 1000 * 10,
  expect: {
    timeout: 10000,
  },
  fullyParallel: true,
  forbidOnly: !!process.env.CI,
  retries: process.env.CI ? 2 : 1
  workers: undefined,
  reporter: 'html',
  use: {
    actionTimeout: 0,
    baseURL: `http://localhost:${PORT}`,
    trace: 'retain-on-failure',
  },
  projects: [
    /* Tried both chromium and firefox {
      name: 'chromium',
      use: {
        ...devices['Desktop Chrome'],
      },
    },*/
    {
      name: 'firefox',
      use: {
        ...devices['Desktop Firefox'],
      },
    },
  ],
  webServer: {
    command: `npm run build && vite preview --port ${PORT}`,
    url: `http://localhost:${PORT}`,
    reuseExistingServer: !process.env.CI,
  },
});

Here is an example of a basic test I have created:
import { expect, test } from 'src/test-utils/playwright';

test('Starting the app shows the search page and the tutorial pop up', async ({
  page,
}) => {
  await page.goto('/');
  await page.getByRole('button', { name: 'Skip' }).click();
  await expect(page.getByPlaceholder('Search')).toBeVisible();
  await expect(
    page.getByRole('button', { name: 'View options' })
  ).toBeVisible();
});

When ever I try to run a test MSW doesn't seem to be intercepting the calls as expected, any idea of why this isn't working? Seems like one of the more basic setups out there.
I have tried changing the URL in the handlers to include http://localhost:3001 (we're using a proxy for our API calls)
From what I can see my setup also looks basically identical to #72's example repo.

I was able to figure out why it wasn't working today, issues caused by my MSAL react calls not being handled when we were in the playwright environment 😓


Encryption - is this possible?

Normal SQLite has extensions/Pager support for adding encryption to the entire DB file i.e. AES ciphers. I was wondering if this project had similar capabilities.
Looking at how the DB is read/parsed, this may be as easy as encrypting/decryption on read/write of chunks.

I recommend applying encryption at the application-level, storing data as blob/string and exposing indexes, as required. Encryption at lower-levels of the stack are a bonus and whilst they add layers around the onion, your application won't be dependent on them.

Someone has done this https://github.com/wireapp/websql
He uses sql.js with encryption enabled.
I've tried this and it worked.
It would be nice if this was also implemented in absurd-sql.

@imannms the repo link is expired. Can you give me some guidance about this problem, please?

https://github.com/hoangqwe159/better-absurd-sql
It is my solution for Encryption. The usage here
import initSqlJs from<EMAIL_ADDRESS>import { SQLiteFS } from "@tashelix/better-absurd-sql";
import IndexedDBBackend from "@tashelix/better-absurd-sql/dist/indexeddb-backend";
import { expose } from "comlink";

/// <reference types="@types/sharedworker" />

// This is an any, as that is the return type from `initSqlJs`
const dbsMap = new Map<string, any>();

const passwordMap = new Map<string, string>();

const dbPath = (name: string): string => `${name}.sqlite`;

let SQL: any;
let isSqlInitialized = false;

async function initializeSql() {
    if (isSqlInitialized) return;

    isSqlInitialized = true;

    SQL = await initSqlJs({ locateFile: (file: any) => file });

    const sqlFS = new SQLiteFS(SQL.FS, new IndexedDBBackend(undefined, passwordMap));

    SQL.register_for_idb(sqlFS);

    SQL.FS.mkdir("/sql");
    SQL.FS.mount(sqlFS, {}, "/sql");
}

async function openDb(dbName: string, password?: string) {
    await initializeSql();

    if (password) passwordMap.set(dbPath(dbName), password);
    if (dbsMap.has(dbName)) return dbsMap.get(dbName);

    const path = `/sql/${dbName}.sqlite`;
    if (typeof SharedArrayBuffer === "undefined") {
        const stream = SQL.FS.open(path, "a+");
        await stream.node.contents.readIfFallback();
        SQL.FS.close(stream);
    }

    const db = new SQL.Database(path, { filename: true });

    db.exec(`PRAGMA cache_size=5000; PRAGMA page_size=8192; PRAGMA journal_mode=MEMORY;`);

    dbsMap.set(dbName, db);

    return db;
}

async function closeDb(dbName: string) {
    const db = dbsMap.get(dbName);
    if (!db) return;

    db.close();
    dbsMap.delete(dbName);
    passwordMap.delete(dbPath(dbName));
}

async function deleteDb(dbName: string) {
    if (dbsMap.has(dbName)) await closeDb(dbName);

    const path = `/sql/${dbName}.sqlite`;

    try {
        (globalThis as any).indexedDB?.deleteDatabase?.(`${dbName}.sqlite`);

        // console.log(SQL);
        // console.log(SQL.FS);

        let exists = true;
        try {
            SQL.FS.stat(path);
        } catch (e) {
            exists = false;
        }

        if (exists) {
            SQL.FS.unlink(path);
        }
    } catch (err) {
        console.error(`Failed to unlink db 'sql/${dbName}.sqlite', reason:`, err);
    }
}

async function dbRun(dbName: string, func: string, ...args: unknown[]) {
    return (await openDb(dbName))[func](...args);
}

async function dbExec(dbName: string, sql: string, params: unknown[] | Record<string, unknown>, password?: string) {
    return await (await openDb(dbName, password)).exec(sql, params, {});
}

// TODO: There is also a db.prepare function, look into using it: https://github.com/jlongster/absurd-example-project/blob/master/src/index.worker.js

// * Exposing the API

const api = { openDb, closeDb, deleteDb, dbRun, dbExec };
export type Api = typeof api;

expose(api);

// biome-ignore lint/suspicious/noGlobalAssign: This is how you initialize shared workers
onconnect = event => {
    const port = event.ports[0];
    expose(api, port);
};


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_adl4jv_0gt/clone/recipe/meta.yaml:23: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_adl4jv_0gt/clone/recipe/meta.yaml:24: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_adl4jv_0gt/clone/recipe/meta.yaml:25: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_adl4jv_0gt/clone/recipe/meta.yaml:26: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_adl4jv_0gt/clone/recipe/meta.yaml:26: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_adl4jv_0gt/clone/recipe/meta.yaml:38: missing_pip_check: For pypi packages, pip check should be present in the test commands
ERROR: /tmp/abs_adl4jv_0gt/clone/recipe/meta.yaml:49: missing_dev_url: The recipe is missing a dev_url
Errors were found


Add readline support to cli

I'm submitting a

[ ] bug report.
[x] feature request.

Current Behaviour:

Currently, you cannot use arrow keys and navigate the input with the CLI, navigate previous commands and other basic CLI actions:

Expected Behaviour:

Use it like any other CLI
This needs some cross-OS testing after importing readline because the dependency to install is a bit different in each OS. Saving some links:
Linux -> Comes by default https://docs.python.org/3/library/readline.html
Windows -> https://github.com/pyreadline/pyreadline
Mac -> https://github.com/ludwigschwardt/python-gnureadline
Do you want to work on this issue?

Yes

I would love to work on this

Please open a PR with your solution and metion this issue :)

This feature can be implemented by using cmd library
I tried it and it works well. here I have attached a  screenshot of the same.
It supports the arrow key gestures and I have used it on Linux and windows.


feat: webwallet and governance framework integration

closes #320

Signed-off-by: sudesh.shetty<EMAIL_ADDRESS>

Note: This change isn’t yet a governance framework integration, but rather an expectation of a credential schema held by agencies making requests to the wallet.

I wonder if we need a feature flag - this feature is incomplete.


Updated references to MUXC

Updated all instances of using:Microsoft.UI.Xaml.Controls to use mux prefix

Description

Searched for all instances of using:Microsoft.UI.Xaml.Controls, ensured they all use the same xmlns:muxc prefix. Updated sample code to also use "muxc".
I anticipated merge conflicts with #208 but they haven't materialized. Perhaps I just haven't waited long enough for the PR to propagate...?
How Has This Been Tested?

manually verified
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Found and resolved the merge conflict. Should be all set now for review and merge.

Philosophical question: should we preface these with "muxc" or "winui"? There doesn't need to be a 1:1 mapping between the alias and the namespace. In fact, the alias is usually a bit more semantic (like "local").

Philosophical question: should we preface these with "muxc" or "winui"? There doesn't need to be a 1:1 mapping between the alias and the namespace. In fact, the alias is usually a bit more semantic (like "local").

I thought we had settled on muxc for our docs. But that may have been wishful thinking on my part because I like the one-to-one symmetry of simply abbreviating Microsoft.UI.Xaml.Controls.
I'm definitely open to feedback on this question.

Philosophical question: should we preface these with "muxc" or "winui"? There doesn't need to be a 1:1 mapping between the alias and the namespace. In fact, the alias is usually a bit more semantic (like "local").

I thought we had settled on muxc for our docs. But that may have been wishful thinking on my part because I like the one-to-one symmetry of simply abbreviating Microsoft.UI.Xaml.Controls.
I'm definitely open to feedback on this question.

I agree with @stmoy on this. Usually one would choose those alias's to be semantic (e.g. contrct7present). Having said that, both "muxc" (which not only stands for the package but partially also for the github url) and "winui" get the point across in my opinion.
In my opinion, from a developer standpoint "muxc" makes more sense however, since "winui" is the product name, but not the technical name (which is Microsoft.UI.Xaml.Controls). So if we think from a developer standpoint, I think it makes a bit more sense to refer to the controls as "muxc" to make a bit more clearer what the origin of those components is. But on the other hand, "muxc" is a bit ambiguous, if one does not know what it stands for, "winui" is pretty self explanatory.


New release on pypi

Hi could you upload a new release on pypi? I'm currently using the github commit in my requirements file since the latest release it's almost 2 years old and does not support python 3

duplicate to #236


Enddat 194

Added a date filter for the processing data workflow step. It is initialized to the union of the selected variables. Functions where added to the WorkflowStateModel and BaseDatasetCollection to support retrieving the union of the date range for the selected variables. Added tests for these functions.

This implements ENDDAT-197, not 194. Forgot to tag my branch appropriately

Closing this pull request as I still need to initialize the date filter correctly.


Deployments

New experience for connecting to various deployment mechanisms (especially GitHub and local Git).

Fixed via these PRs:
https://github.com/Microsoft/vscode-azureappservice/pull/697
https://github.com/Microsoft/vscode-azuretools/pull/314
https://github.com/Microsoft/vscode-azuretools/pull/319
https://github.com/Microsoft/vscode-azuretools/pull/321


schedule: fix panic caused by removing tombstone (#2015)

What problem does this PR solve?
cherry-pick #2015 to release-3.1 branch
What is changed and how it works?
This PR fixes the panic problem by resetting the attached function when removing the store limit.
Check List
Tests

Unit test

Related changes

Need to be included in the release notes

Codecov Report

Merging #2038 into release-3.0 will decrease coverage by <.01%.
The diff coverage is 100%.

@@               Coverage Diff               @@
##           release-3.0    #2038      +/-   ##
===============================================
- Coverage        69.44%   69.43%   -0.01%
===============================================
  Files              170      170
  Lines            16768    16769       +1
===============================================
  Hits             11644    11644
- Misses            4144     4145       +1
  Partials           980      980

Impacted Files
Coverage Δ

server/schedule/operator_controller.go
87.19% <100%> (+0.47%)
:arrow_up:

server/util.go
68.15% <0%> (-4.46%)
:arrow_down:

server/region_syncer/client.go
78.94% <0%> (-3.95%)
:arrow_down:

server/leader.go
77.95% <0%> (-2.45%)
:arrow_down:

server/handler.go
52.11% <0%> (+0.49%)
:arrow_up:

server/schedule/operator.go
87.58% <0%> (+0.64%)
:arrow_up:

server/namespace_cluster.go
85.71% <0%> (+3.89%)
:arrow_up:

server/schedulers/shuffle_hot_region.go
65.38% <0%> (+6.41%)
:arrow_up:

Continue to review full report at Codecov.

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

/run-all-tests


[Feature Request] Spline arrows & lines like excalidraw.

I hate to compare tldraw and excalidraw. But I am a heavy ex excalidraw user.
It would be great if we can make spline arrows & lines as in excalidraw, with multiple control points to adjust the curve.
Currently tldraw only supports circular arrows, which are okay for most of the use cases, but when making flowchart like diagrams, it falls short.
Please add an ability to add multiple waypoints to adjust the curves in arrows and lines.
Also an ability to do it on the go, by single clicking the starting point and producing the curves wherever next click happens. (Just as in excalidraw)

Hey you can do this with lines currently, but it would be good to be able to do this with arrows too!
You can drag out additional handles in lines, or hold the shift-key while drawing them.


Demote and drop not promoted transactions when txpool overflow

Description
The future (nonce) transactions would fulfill the queue, but no transaction could not be promoted when txpool overflow. Then txpool would reject all transactions, which is vulnerable when attackers send as many future transactions as possible.
Since the future transactions would not be executed, this type of attack can be done without any cost at all.
The PR would demote all future transactions in addtx if the txpool overflow, when the account demote count reaches the maximum limit, drop all its transactions to give more space.
See the new test in PR which simulate how this feature brought the overflow txpool back to normal.
Changes include

[ ] Bugfix (non-breaking change that solves an issue)
[ ] Hotfix (change that solves an urgent issue, and requires immediate attention)
[x] New feature (non-breaking change that adds functionality)
[ ] Breaking change (change that is not backwards-compatible and/or changes current functionality)

Checklist

[x] I have assigned this PR to myself
[x] I have added at least 1 reviewer
[x] I have added the relevant labels
[ ] I have updated the official documentation
[x] I have added sufficient documentation in code

Testing

[x] I have tested this code with the official test suite
[ ] I have tested this code manually

Another similar PR #105 has merged.
Close this legacy PR.


Tweak stack.yaml snapshot to include katip-<IP_ADDRESS>

See input-output-hk/cardano-haskell#20
Apologies for the rebuild.

bors r+
(bors doesn't "see" commands inside review comments)


Add OpenAIMultiFunctionsAgent to import list in agents directory

Added OpenAIMultiFunctionsAgent to the import list of the Agents directory
@hwchase17

thanks @Tajcore!


Esri Geoportal Server 2.6.1 - extending predefined metadata profiles and compatibility the INSPIRE profile in accordance with new version of Technical Guidance

I have two issues that I would like to express.
During delivering Geoportal Server Catalog 2.6.1  for one of our most important clients we faced problems with extending  predefined metadata profiles and with compatibility  the INSPIRE profile for datasets and service in accordance with the new version of the Technical Guidance for the implementation of INSPIRE dataset and service metadata based on ISO/TS 19139:2007.

Problems with extending predefined metadata profiles

As a guideline we are using example published on github
https://github.com/Esri/geoportal-server-catalog/tree/master/geoportal/src/main/webapp/app/gxe/types/myprofile/
but this example handles only situation of manipulating existing  elements or attributes throughout inherited methods defined in
MyProfileDocumentType.js i.e:
•	beforeInitializeElement
•
•	afterInitializeElement
•
•	beforeInitializeAttribute
•	afterInitializeAttribute
We need to fulfill requirement to add new element to xml other than defined in
https://github.com/Esri/geoportal-server-catalog/blob/master/geoportal/src/main/webapp/app/gxe/types/myprofile/base/templates/DataRoot.html
our purpose is to add two additional elements into CI_OnlineRsource element
gmd:distributionInfo
gmd:MD_Distribution
gmd:transferOptions
gmd:MD_DigitalTransferOptions
gmd:onLine
gmd:CI_OnlineResource
gmd:linkage
gmd:URLhttp://adres_do_uslugi_wms/?REQUEST=GetCapabilities&SERVICE=WMS</gmd:URL>
</gmd:linkage>
gmd:name
gco:CharacterStringWMS service</gco:CharacterString>
</gmd:name>
gmd:description
gco:CharacterStringWMS service details</gco:CharacterString>
</gmd:description>
gmd:function
<gmd:CI_OnLineFunctionCode codeSpace="ISOTC211/19115" codeList="http://www.isotc211.org/2005/resources/Codelist/gmxCodelists.xml#CI_OnLineFunctionCode" codeListValue="information">information</gmd:CI_OnLineFunctionCode>
</gmd:function>
</gmd:CI_OnlineResource>
</gmd:onLine>
</gmd:MD_DigitalTransferOptions>
</gmd:transferOptions>
</gmd:MD_Distribution>
</gmd:distributionInfo>
What have we done to achieve this result:

copied standard iso 19115 definition of gmd/Distrubtion into
C:\apache-tomcat-8.5.37\webapps\
renamed files - adding prefix 'My' to distinguish from original implementation
updated references to js and html files
Add gmd:name and gml:description nodes to  C:\apache-tomcat-8.5.37\webapps\geoportal\app\gxe\types\myprofile\gmd\distribution\templates\MyTransferOptions.html
Updated reference in C:\apache-tomcat-8.5.37\webapps\geoportal\app\gxe\types\myprofile\base\DataRoot.js
and C:\apache-tomcat-8.5.37\webapps\geoportal\app\gxe\types\myprofile\base\templates\DataRoot.html
We modified dojoConfig to be sure that new files were loaded properly

After all these steps we got an error:
description"uniqName_15: parser returned unfilled promise (probably waiting for module auto-load), unsupported by _WidgetsInTemplateMixin.   Must pre-load all supporting widgets before instantiation."
We found a workaround for this situation by not referring our own classes but to original one by override implementation "on the fly"
But then we are getting another error "multipuleDefine for "https://js.arcgis.com/3.22/esri/dijit/metadata/types/iso/gmd/distribution/Distribution.js"
and it causes a situation when every Distribution node in all iso depended profiles in our catalog gets custom implementation.
Can you provide us some instructions what we are doing wrong, some working example with adding/removing whole elements, or how to modify more sophisticated metadata profile i.e Inspire.
We can provide live presentation of our problems

Problems with compatibility  the INSPIRE profile for datasets and service in accordance with the new version of the Technical Guidance for the implementation of INSPIRE dataset and service metadata based on ISO/TS 19139:2007.

For example:

in some cases, an INSPIRE metadata element is mapped to a free text element in ISO 19139, but these TGs recommend or require the use of code list value through an gmx:Anchor element
TG Recommendation C.8: metadata/2.0/rec/common/use-anchors-for-cv-keywords
If the keywords from controlled vocabularies are used and the individual keywords have a specified canonical URI within that controlled vocabulary, these keywords should be encoded using the gmd:keyword/gmx:Anchor element. The xlink:href attribute of the gmx:Anchor element should be used for referring to the canonical URI of the keyword.

At the moment Esri Geoportal Server doesn’ t support this mechanism.

the following element is not correctly implemented: limitation on public access (other constraints)

In accordance with TG the domain of this element shall be the following code list:
http://inspire.ec.europa.eu/metadata-codelist/LimitationsOnPublicAccess
see: TG Requirement C.18: metadata/2.0/req/common/conditions-for-access-and-use and C.2.22 Limitations on public access
Are you planning to adapt this product to the new technical guidelines?

Sounds like work that can be used by many. Can I suggest that you pull tree, create a branch do a local build, and then submit a patch
skip the My, and just modify to the files to the spec, and submit changes.
Do in small steps, one change, one commit.
When I see something like:
uniqName_15: parser returned unfilled promise
my experience says:

did my change cause the javascript file to be invalid, or the template to be invalid
did I add or take away from the declared modules, and did I add the object reference in the right place. Order is important.
did I misspell a template name, or copy a file and not change the template.


Move IREE to be GitHub first

As announced in https://groups.google.com/d/topic/iree-discuss/F07vsG9Ah4o/discussion
IREE will be trying out a flow where we merge directly on GitHub and also have a separate branch that tracks the internal Google source repository. Frequent merges between branches will be used to propagate changes.
This will allow us to hide some of the complexity of syncing with our internal source repository and get it out of the way of everyday developer flows.
Internal tracking bug http://b/159492658

Migration is complete and (somewhat) stable.

Should we delete the old master branch before calling this done?
On Mon, Aug 3, 2020, 17:20 Scott Todd<EMAIL_ADDRESS>wrote:

Migration is complete and (somewhat) stable.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/google/iree/issues/2279#issuecomment-668303208, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABLXN6FOBYE7S6QXLYH4TU3R65H6PANCNFSM4OFDMVYA
.

Should we delete the old master branch before calling this done?

We should do that regardless :P

Deleted


feat: fix decidableEq and decidableLT fields in the LinearOrder instance on Bool

Currently, the fields are filled in by an automatic parameter, which does not match the canonical decidability instances on Bool, creating some diamonds down the road.

bors r+


Create Your CockroachCloud Cluster Doc Update

Re: Create Your CockroachCloud Cluster
Issue Description
sizing example could use an estimate on the index strategy as well as online schema changes
Suggested Resolution
when users report large numbers of indexes, it throws the sizing calculation off and we need a better example in that case. For schema changes, when new version of a large table is created, we need to account for the double of the amount of space needed to perform an online schema change.

cc: @jseldess

@kannanlakshmi, @Amruta-Ranade, can you take a look at this in your next grooming/prioritization?

Closing - as this is no longer relevant (sqlalchemy steps are no longer included on this doc)

Closing in favor of CC-3704


Add the definitions for solr4 external as they are called from scenarios with external databases.

Signed-off-by: Prajakta Purohit<EMAIL_ADDRESS>

Pipeline Information -
Adhoc - https://buildkite.com/chef/chef-chef-server-main-omnibus-adhoc/builds/3625


Object inside template header

How do I access the object inside the loop in a template header? I would like to manipulate the story data.  Thx

version 0.9.3 added this feature
you can do it like this
<template>
  <Stories :stories="items">
    <template #header="{story}">
      <story-header :story="story" />
    </template>
  </Stories>
</template>


Basic tutorial

Adds a simple tutorial start-to-finish tutorial for DaCapo.
Still a few things left to resolve before its ready for merging:

[ ] Loss behaving strangely after the first validation (temporary solution involves giving an extra singleton dimension to the provided data)
[ ] Validation fails (we want to show validation loss/scores, validation outputs across iterations, and how to finally take the best iteration and apply it with the post processing to a volume manually)
[ ] stdout too verbose
[ ] plotting functions should be built into DaCapo (there are plotting functions, not sure how to include bokeh plots so I didn't dive too deep into this)

I can't modify in your fork. so, i created a new branch basic_tutorial, i will merge your change to that branch. and i will add the missing parts there before merging to main


Add a dedicated RapidsConf option to tolerate GpuOverrides apply failures

This PR provides an option to tolerate GPU plan override failures (off by default) without having to toggle spark.rapids.sql.enabled before and after a problematic query.
Closes #3739
Signed-off-by: Gera Shegalov<EMAIL_ADDRESS>

build


Fix #272 PlugUpdate and delayed loading

Force syntax files sourcing on lod_ft.

Only negative to this fix is sometimes double sourcing syntax files, not a big deal it seems.

Thanks!

@junegunn No problem. :)


Please add a link to this repo in the documentation page

Thanks!

Done. Will be available in few minutes.

Done. Will be available in few minutes.


 This isn't so bad, but it can be far slower on a slower or already-busy computer:
https://user-images.githubusercontent.com/25163139/203137381-9ba3fc55-bfb2-40a0-903f-a0f52b6eeec0.mp4
Wondering if there are any quick wins here. The slight delay makes the whole experience feel sluggish and old.
@esonnino

Moving this to next milestone as part of issue tracking and cleaning for the current milestone as we are close to the release.

We updated webviews to be reused instead of recreating for each step (which have svg/markdown media). https://github.com/microsoft/vscode/pull/186267


Adding an Element with no groupBy in MapStoreThrows NPE

Null checks should be added to the for loops within the AddElementsHandler to stop NPE's being thrown if there is no groupBys.

Merged into develop


ribs: allow deleting RIBs

We'll have to make sure this isn't confused with "updating with a new RIB" once we start payments, because we want to trace the history of the RIBs they've used.

Closes #275


Add cgroups filesystem interface

This is the clean cgroups interface from https://github.com/intel/cri-resource-manager/pull/639 with these additions:

cgroupsblkio ported on top of the new interface
unit tests added
fsi/fsios/fsimock added: abstract filesystem interface, native (pass-through) filesystem implementation, and mock filesystem implementation, respectively

This draft was replaced by PR #29 .


Pause / resume all.

This would be handy for a number of scenarios, including pausing games / experiences, and dealing with background tabs. Ideally it would flag sounds so as to resume only appropriate sounds (ie. not sounds that were paused independently).

This would be really useful


Change some hard coded path separators to path.sep

This is so it will run on windows

Good change, someone else mentioned this issue to me recently. Tested on Windows and confirmed working. Thanks!


[WT-1]: Feature/add trash

Added Trash functions
Merged with master
Fixed tests
Updated package.json

Codecov Report

Merging #107 (37f5865) into master (d972dd6) will increase coverage by 0.23%.
The diff coverage is 86.79%.

@@            Coverage Diff             @@
##           master     #107      +/-   ##
==========================================
+ Coverage   76.18%   76.42%   +0.23%
==========================================
  Files          38       39       +1
  Lines         802      844      +42
  Branches      103      106       +3
==========================================
+ Hits          611      645      +34
- Misses        179      187       +8
  Partials       12       12

Impacted Files
Coverage Δ

src/drive/storage/types.ts
100.00% <ø> (ø)

...ve/storage/mothers/folderContentResponse.mother.ts
100.00% <ø> (ø)

src/shared/headers/index.ts
73.07% <42.85%> (-16.40%)
:arrow_down:

src/drive/share/index.ts
86.27% <80.00%> (-4.21%)
:arrow_down:

src/drive/index.ts
100.00% <100.00%> (ø)

src/drive/storage/index.ts
98.03% <100.00%> (ø)

src/drive/trash/index.ts
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Add post about the incoming Salta sprint

This looks good!  Thanks @jmloyola.

cc:  @symeneses
There are a couple of key links in this blog about the upcoming scikit-learn sprint, as part of SciPy Latin America in September 2022.

@jmloyola in your sprint repo (https://github.com/jmloyola/sklearn-sprint-argentina-2022):

Is there info about pair programming (I have only 1 year of Spanish so can read some, but not all Spanish).
Should we add an intermediate level issue for participants who may be ready for it? such as this one:
https://github.com/scikit-learn/scikit-learn/issues/23462
If scikit-learn office hours will return in September, we can add a note that folks can follow-up and connect with the scikit-learn team after the sprint event to continue their work.

@jmloyola in your sprint repo (https://github.com/jmloyola/sklearn-sprint-argentina-2022):
1. Is there info about pair programming (I have only 1 year of Spanish so can read some, but not all Spanish).

2. Should we add an intermediate level issue for participants who may be ready for it? such as this one:
   [Make all estimators use `_validate_params` scikit-learn#23462](https://github.com/scikit-learn/scikit-learn/issues/23462)

3. If scikit-learn office hours will return in September, we can add a note that folks can follow-up and connect with the scikit-learn team after the sprint event to continue their work.

Yes, there is a paragraph with an overview of the driver-navigator pair-programming technique. There is also a link to this blog post explaining more about the topic.
Yes :D Though I think most participants will be making their first contributions to open-source, thus the proportion of easy issues should be bigger.
Good idea. I can link to the scikit-learn calendar and add some information about the office hours. If it is OK, when we approach the date of the event, I can share the link to join the scikit-learn discord (I think it has a lifespan of five days).

@jmloyola
cc:  @symeneses
In your instructions repo, is there information/link about how folks can join the scikit-learn Discord server?

In your instructions repo, is there information/link about how folks can join the scikit-learn Discord server?

No yet. I think we have to wait to share the link to join the scikit-learn Discord server. The shared link expires in seven days, thus I'll share it the week before the sprint.


Commit /subscriptions, /services and /rebuildserviceset API endpoints

/subscriptions shows the subscription details (tokens/regids, push
service type, service names) of a given subscriber. If no service is
provided in the query, all services are queried.
As part of this change, the set of all service names must be tracked in
redis. KEYS would be too slow to use, so a new set SERVICES.{0} was added.

If the redis databases sharded, then the {0} could be used as a shard scheme,
to avoid hitting multiple databases for what would be a small set.
/rebuildserviceset is added, to be manually run to create the service
set for pre-existing uniqush users. This hasn't been fully tested yet.

For convenience, /psps was also added.

The API response maps service names to a list of psps with that service name.
(From the documentation, I'm not sure if it is meant to be possible
for a service name to be shareable among 2 or more PSPs, e.g. with
different push service types.)

Cool! Yeah, I was wondering of an easy way to provide users a way to migrate to have the SERVICES{0} set. I was also thinking other people using twemproxy could use a different set of chars other than curly brace for the key, but these are all relatively minor, solveable problems. At work I ended up creating this set by hand initially.
Yeah we only need one of these sets on a sharded Redis DB, and, without an explicit key, it's much less clear which partition it ends up on. Our twemproxy is configured to take the md5 of the key -- if it's not enclosed in curly braces then it will take the md5 of the entire key. Fortunately nothing doing any cross-set intersections or the current schema may have given us trouble with our sharded setup.
@monnand Would like your input. Also a clarification -- is a service name meant to be shareable amongst 2 or more PSPs, esp with different push service types?

Note this addresses issue #98

@TysonAndre merge conflicts

@mishan fixed merge conflicts

:+1: thanks!


Suggestions: Info command and a message

When opening a protection that is locked by someone, it will send a message saying: "You are opening a chest/furnace locked by [player name]"
Running /keys info and punching a protection will tell a user who locked it and the people that were added to use it.

I can see this being very useful. showing all flags and people attached to it. (when we get flags for lockable stuff)


Makes user compaction metadata independent.

Before this change user initiated compactions stored config in a single place in zookeeper and had a single counter in each tablet.  This system made it impossible for concurrent user initiated compactions to run on the same table with different configuration.  This commit adds compaction config storage and tablet metadata that has 1:1 correspondence with the fate operation driving a user initiated compaction.  This allows each user initiated compaction to be independent.  This new system also avoids some odd race conditions with tablet metadata updates that existed with old per tablet compaction counter.  Instead of a counter each tablet now conceptually has a set of the fate transaction ids that have completed a compaction.
See the new documention in TableOperations.compact() for an example.
fixes #3517

All tests in CompactionIT are passing with these changes.

Merged this because I have another PR that builds on this to remove code from the tserver and I need to get that change in so I can start making changes to improve the tablet server code.  If anyone was looking at this feel free to comment still I can address any comments in follow on commits or issues.


Use async InstanceTracker methods in SchedulerActor

Releated to #4802, #3031, #4693

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

ok to test

thanks for the PR! maybe some conflicts with https://phabricator.mesosphere.com/D317
want to rebase?

Rebased


Isolated scope

Hello @dennybiasiolli @NicoBurno, sorry for the long delay!
As I've seen no progression on this ( no pun intended! ), and I had a Torino OpenSource Saturday to spend on this, here is my implementation for the isolated scope.
This is a full refactor of the previous implementation :) All features are preserved ( apart custom http headers ).
The refactor is based on a factory object, which creates an object that can be used to manipulate the viewer status.
Please have a look and ask questions!
The NgPdfFactory should be more appropriately named NgPdfViewer.
Thanks!

@*dennybiasiolli *as this is a complete new version, may I propose to
remove phantomjs and instead use the new chrome headless?

@endorama yes, it's a good idea


Node attribute storage in nodes

Stores node attributes terminal,stuck, and vacuous in nodes rather than in KCFG or KCFGExploration.

adds attr field to KCFG.Node
adds enums NodeAttr (base), KCFGNodeAttr, and KCFGExplorationNodeAttr
adds add_attr, remove_attr, and discard_attr to kcfg for updating the attrs of a given node, which replaces the node with a new one, since nodes are immutable.
replace_node now takes a node. let_node is now for updating specific fields of nodes.
Checks terminal, stuck, and vacuous by reference to the attributes on the node
Removes separate terminal, stuck and vacuous lists.
from_dict and to_dict on Node and KCFG updated to store relevant attributes in the node dicts. KCFGStore translates to and from the existing on-disk format.
Tests in test_proof.py previously using read_proof and write_proof are now using read_proof_data and write_proof_data, since the older way of reading and writing does not use the KCFGStore and relies on KCFG.to_dict directly, without the translation.

Do we also need to add the refuted attribute? I added it to the list displayed by show.py in PR #877, but it's possible that there were other places where I should have added it.

Do we also need to add the refuted attribute? I added it to the list displayed by show.py in PR #877, but it's possible that there were other places where I should have added it.

@lucasmt I guess we aren't defining node attributes by the ones which are printed in the KCFGs. We're trying to just have internal node attributes here where the attribute could be determined just by looking at the node, i.e. not the rest of the CFG or Proof.


bake: handle git auth token when parsing remote definition

needs https://github.com/docker/actions-toolkit/pull/285

We got the following output when trying to build from a remote bake definition on a private repo: https://github.com/docker/test-docker-action/tree/remote-private
$ docker buildx bake https://github.com/docker/test-docker-action.git#remote-private
#0 building with "default" instance using docker driver

#1 [internal] load git source https://github.com/docker/test-docker-action.git#remote-private
#1 0.225 fatal: could not read Username for 'https://github.com': terminal prompts disabled
#1 ERROR: failed to fetch remote https://github.com/docker/test-docker-action.git: git error: exit status 128
stderr:
fatal: could not read Username for 'https://github.com': terminal prompts disabled

------
 > [internal] load git source https://github.com/docker/test-docker-action.git#remote-private:
0.225 fatal: could not read Username for 'https://github.com': terminal prompts disabled
------
ERROR: couldn't find a bake definition

Using a GIT_AUTH_TOKEN secret doesn't seem to work:
/usr/bin/docker buildx bake https://github.com/docker/test-docker-action.git#remote-private --set *.secrets=id=GIT_AUTH_TOKEN,src=/home/runner/work/_temp/docker-actions-toolkit-Rrz5jT/tmp-1910-aMBJe65r7Kf4 --print default
#0 building with "default" instance using docker driver
#1 [internal] load git source https://github.com/docker/test-docker-action.git#remote-private
#1 0.039 Initialized empty Git repository in /var/lib/docker/overlay2/l763ozu3v99vupy2gmketmi8t/diff/
#1 0.279 fatal: could not read Username for 'https://github.com/': terminal prompts disabled

which makes sense as loading remote bake definitions is done by a dedicated solve request, not during build: https://github.com/docker/buildx/blob/8abef5908705e49f7ba88ef8c957e1127b597a2a/bake/remote.go#L62
So would need to attach this secret to the session. UX wise I'm not sure what we could do. Using an override with --set is not correct so best guess is we need a new flag to bake cmd.

Opened https://github.com/docker/buildx/pull/2363 issue and https://github.com/docker/buildx/pull/2363 to implement git auth token.


Added sections on the landing page

Added features, about and contribute sections on the landing page
Content was taken from readme.md.
Working demo link:  https://dash-three.vercel.app/.
Give feedback on the design and dontent of the added sections.

Requesting a review from @sauravhiremath to review page content and check the code in general.

I ran prettier and removed unused assets from old commits.

So, i need to remove the styled css import and import the css normally, changing the className value from {styled.class} to "class".
Also i have added latitudes and longitudes using content property of the pseudo element so should i change it entirely into unicode or only spaces in unicode.

Yeah, just importing CSS as you mentioned
Just the apostrophes and degrees in Unicode, like ", ' and °. No need to encode spaces


Expenses tracker

So, the loot tracker is an amazing and useful feature in runelite. I use it a lot to calculate the profits of my slayer/bossing.
But i think there is one aspect missing. Some bosses/tasks require a lot of supplies which makes it difficult to know the real profit. That's why i suggest a runelite feature that, along with your loot, tracks your expenses. For example, it would calculate the costs of Arrows/bolts, potions, food, runes etc.. consumed.
Any feedback is appreciated.

duplicate of #6700


Two new functions in HTMLNode.swift and one variable in HTMLParser.swift

Hello
In HTMLNode.swift add two new functions:
-findChildTagAttr : find HTMLNode with a tag name, html atribute name (p.e. id,class,style, etc) and attribute value.
-findChildTagsAttr : same as above, except that find a HTMLNode array
In HTMLParser.swift added a string called htmlString for store webpage as string.
Thanks for developing this library.
Best Regards

Merged, thanks PR.
Please clean PR next time :)


Locally installed vector db options?

I have a question around storing the dreams. For a small POC project would it be possible to use a locally hosted vector db rather than Pine Cone? Perhaps something like Elasticsearch or other vector capable choices?

Great question! We have this in our backlog. In the mean time, you can replace our https://github.com/DivergentAI/dreamGPT/blob/main/dreamgpt/store/store.py with your custom logic. Should be quite straight forward.


Fixed #28925 -- Made durations-only expressions work on SQLite and MySQL.

https://code.djangoproject.com/ticket/28925

Sergey, did you notice the test failures?

@timgraham, yes, of course :slightly_smiling_face:. Still working on the fix.

Please reopen if you return to it.


missing function definition.

while moving the code to under WMCore/Services/TagCollector
Pylint complains about
E0602, line 41 in xml_parser: Undefined variable 'adjust_value
https://github.com/dmwm/WMCore/blob/master/src/python/WMCore/ReqMgr/Utils/utils.py#L68
@vkuznet, Valentin, is there a missing code in original code above?

Seangchan,
this code was originally copied from DAS codebase where I developed xml parser
for CMS XML docs. The adjust_value function is defined here:
https://github.com/dmwm/DAS/blob/master/src/python/DAS/utils/utils.py#L587
with float_number_pattern defined here:
https://github.com/dmwm/DAS/blob/master/src/python/DAS/utils/regex.py#L56
and int_number_pattern defined here:
https://github.com/dmwm/DAS/blob/master/src/python/DAS/utils/regex.py#L58
So, I'll suggest to take these pieces and embed them into new TagCollector.
I'll avoid adding dependencies on DAS since it will create a circular
dependency between WMCore and DAS.
Can you do it or do you want me to do it for TagCollector code?
For later I think you first need to put TagCollector code into WMCore master
head and then I can fix it.
Best,
Valentin.
On  0, ticoann<EMAIL_ADDRESS>wrote:

while moving the code to under WMCore/Services/TagCollector
Pylint complains about
E0602, line 41 in xml_parser: Undefined variable 'adjust_value
https://github.com/dmwm/WMCore/blob/master/src/python/WMCore/ReqMgr/Utils/utils.py#L68
@vkuznet, Valentin, is there a missing code in original code above?
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/dmwm/WMCore/issues/7488

Thanks Valentin, I will update the code.

code is updated in #7484, closing this issue


USB devices don't autoremove and repopulate

Summary
When the automated USB passthrough occurs, it doesn't remove the previous USB drives. This is typically not a problem, unless you unplug one of the automated pass-throughed USB drives. If you do that, the VM fails to start.

Fixed.


Add 1.19.4 and 1.20.1 Parchment version badges

 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.

Currently, this is on hold pending the reconfiguration of the website's deployment build configs. The CI agents are incompatible with with website's build configurations, as the new CI agents are incapable of running Docker images (to my understanding).
I'm not sure as to the state of the website's infrastructure to fix those build configs, so I'm deferring to @marchermans.


Draft: Add Environment Symbols registration to Power Fx Interpreter

We want a path to add symbols specific to a host's context. This adds an immutable store, and methods on RecalcEngine for updating that.
When the symbol context changes, all expressions in the RecalcEngine must be rebound and evaluated with the new context, which is why we've taken an immutable approach to these updates.
NOTE: Don't review in depth yet. There'll  be a refactor PR of Option Sets that needs to get checked in before this becomes useable.

Example snippet of use:
var optionSet = new OptionSet("OptionSet", new Dict<string, string>(){
{ "Option1", "option_1" }
{ "Option2", "option_2")
});

var recalcEngine = new RecalcEngine(settings);
recalcEngine = recalcEngine.WithEnvironmentSymbol(optionSet);

var context = new RecordType.AddField("OptionSetField", new OptionSetType(optionSet))

result = recalcEngine.Eval("If(OptionSetField = OptionSet.Option1, \"matches 1\", \"doesn't match 1\")", context,...)


Could not find module ‘IHaskell.Display.Demo’

Built IHaskell with stack according to the instructions) and if I then run:
stack exec jupyter -- notebook

I get:
[I 09:45:09.848 NotebookApp] Serving notebooks from local directory: /home/erikd/Git/Haskell/ihaskell-demo
[I 09:45:09.848 NotebookApp] 0 active kernels
[I 09:45:09.848 NotebookApp] The Jupyter Notebook is running at:
[I 09:45:09.848 NotebookApp] http://localhost:8888/?token=347fe1039819e1b0660527461fbc82ebdc6b00fab4fe752c
[I 09:45:09.848 NotebookApp] Use Control-C to stop this server and shut down all kernels (twice to skip confirmation).
[C 09:45:09.849 NotebookApp]

    Copy/paste this URL into your browser when you connect for the first time,
    to login with a token:
        http://localhost:8888/?token=347fe1039819e1b0660527461fbc82ebdc6b00fab4fe752c
[I 09:45:10.705 NotebookApp] Accepting one-time-token-authenticated connection from ::1
[W 09:45:13.461 NotebookApp] Notebook first.ipynb is not trusted
[I 09:45:13.690 NotebookApp] Kernel started: 62a97285-b025-42f0-a830-ab7e220da7a8
[W 09:45:13.791 NotebookApp] 404 GET /static/components/codemirror/mode/ihaskell/ihaskell.js?v=20180827094509 (::1) 4.50ms referer=http://localhost:8888/notebooks/first.ipynb
ihaskell: Could not find module ‘IHaskell.Display.Demo’
Perhaps you meant
  IHaskell.Display.Plot (from ihaskell-plot-<IP_ADDRESS>)
  IHaskell.Display (from ihaskell-<IP_ADDRESS>)
Use -v to see a list of the files searched for.

I get this whether I open an existing Haskell notebook or create a new Haskell notebook. It does not happen when starting a new Python3 notebook.

No, its a different repo. Does it need to be the same?

The weird thing is that neither IHaskell nor my project have a module named IHaskell.Display.Demo

Oops, this is a different issue than I thought it might be. As you mentioned, I can't see any references to "IHaskell.Display.Demo" anywhere in the source code. However, the name of your repo matches the ihaskell-* naming scheme for the IHaskell display modules. What happens if you rename it to e.g. foo and try launching an IHaskell noteook. Do you still see this behaviour?

Ah!
Renaming the directory alone did nothing, renaming the cabal file and the project name inside the cabal file got me past that problem only to run into a different problem.
I don't know about anyone else, but it i was going to be doing something in IHaskell, the first thing I would be doing is creating a git repo and the obvious name would "ihaskell-. Surely this is something that could and should be fixed.

I think I've found where this is happening. Essentially IHaskell loads all (support) libraries that are prefixed with ihaskell-. This can be surprising but also seems reasonable, and I can't think of a better approach. I have no plans to change this behaviour, and am closing this issue. If there is a different approach you think would work better in general, feel free to reopen.